Move one simplification call in lowering. - #9355
Conversation
There was a problem hiding this comment.
This should be moved up above the now-final simplification
There was a problem hiding this comment.
Andrew argued that the string should change, instead of moving this. @abadams
There was a problem hiding this comment.
Suggest moving it after the custom passes and changing string to "Conceptual stmt:", because it's what you get if you compile to conceptual_stmt.
There was a problem hiding this comment.
Named it:
debug(1) << "Lowering after reaching conceptual Stmt:\n"
<< s << "\n\n";As Claude is grepping for "Lowering after" to find the different sections of the lowering phase.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9355 +/- ##
==========================================
+ Coverage 70.08% 70.10% +0.01%
==========================================
Files 259 260 +1
Lines 79158 79295 +137
Branches 19293 19328 +35
==========================================
+ Hits 55477 55587 +110
- Misses 17886 17927 +41
+ Partials 5795 5781 -14 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Deadlocks exposed here fixed by #9372 |
34b2370 to
84cc3b3
Compare
|
Waiting for #9372 to land to update this branch, but this seems to work now. @abadams please take note of Claude's initiative to disable a lower pass in LLVM 21, due to a bug fixed in 22 regarding SVE on ARM. I'll open a PR separately for this LLVM pass skip and remove the skips in all the tests to see if that makes the test suite pass. |
|
Looks like it needs to be rebased. A bunch of unrelated changes are in it now. |
Co-authored-by: Andrew Adams <andrew.b.adams@gmail.com>
… conceptual stmt'
00a65a3 to
a00b0ef
Compare
When debugging some stuff not part of this PR, I spotted a missing simplification, which Andrew determined to be AFTER hoist_loop_invariant_xxx. This moves the simplification in the right spot.
Breaking changes
I hope none, let's see.
These do not necessarily disqualify a PR from being merged, but they should at
least be tagged with the
release_noteslabel.Checklist