Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multiprecision: ignore buggy -Waggressive-loop-optimizations in subtraction #334

Merged

Conversation

ioxid
Copy link
Contributor

@ioxid ioxid commented Feb 17, 2025

No description provided.

@ioxid ioxid force-pushed the fix-multiprecision-aggressive-loop-optimizations-subtract branch from a0b1118 to 1cd2d77 Compare February 17, 2025 12:26
Copy link

Clang Test Results

  174 files  ±0    174 suites  ±0   16m 45s ⏱️ + 1m 3s
8 545 tests +4  8 539 ✅ +4  6 💤 ±0  0 ❌ ±0 
8 611 runs  +4  8 605 ✅ +4  6 💤 ±0  0 ❌ ±0 

Results for commit 1cd2d77. ± Comparison against base commit 0ec44ed.

Copy link

Gcc Test Results

  174 files  ± 0    174 suites  ±0   15m 5s ⏱️ +5s
8 448 tests  - 93  8 442 ✅  - 93  6 💤 ±0  0 ❌ ±0 
8 514 runs   - 93  8 508 ✅  - 93  6 💤 ±0  0 ❌ ±0 

Results for commit 1cd2d77. ± Comparison against base commit 0ec44ed.

This pull request removes 99 and adds 6 tests. Note that renamed tests count towards both.
pack_equal_test_suite ‑ bubb_to_bubb_1
pack_equal_test_suite ‑ bubb_to_bubb_2
pack_equal_test_suite ‑ bubb_to_bubb_3
pack_equal_test_suite ‑ bubb_to_bulb_1
pack_equal_test_suite ‑ bubb_to_bulb_2
pack_equal_test_suite ‑ bubb_to_lubb_1
pack_equal_test_suite ‑ bubb_to_lubb_2
pack_equal_test_suite ‑ bubb_to_lubb_3
pack_equal_test_suite ‑ bubb_to_lulb_1
pack_equal_test_suite ‑ bubb_to_lulb_2
…
pack_imploder_test_suite ‑ be_to_be_1
pack_imploder_test_suite ‑ be_to_be_2
pack_imploder_test_suite ‑ be_to_le_1
pack_imploder_test_suite ‑ be_to_le_2
pack_imploder_test_suite ‑ be_to_le_3
pack_imploder_test_suite ‑ bubb_to_lulb_4

@ioxid ioxid requested a review from martun February 17, 2025 13:17
@martun martun added this pull request to the merge queue Feb 17, 2025
Merged via the queue into master with commit b95c1b9 Feb 17, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants