Add -Zautodiff_post_passes flag to limit which llvm passes to run after enzyme to make autodiff tests more robust#156777
Conversation
64371d7 to
0da69f6
Compare
|
I tested this PR still works after enzyme submodule update: #157716 |
|
I think I've seen some of these changes before, so my concern is that either we continue to make the tests increasingly weaker, or we'll keep swapping back and forth between different outputs. We probably want to remove |
04bcb8e to
08f1ff1
Compare
|
Thanks! |
a7089cb to
bd0eab1
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
lgtm, but can you clean up the story a bit? Maybe make one commit first to introduce the infra, and then another one to update the test. |
85ca87b to
2a9e45d
Compare
|
@bors r=ZuseZ4 |
…iff-2, r=ZuseZ4 Add -Zautodiff_post_passes flag to limit which llvm passes to run after enzyme to make autodiff tests more robust * Add -Zautodiff_post_passes flag to limit which llvm passes to run after enzyme to make autodiff tests more robust * Change some llvm ir check in codegen_llvm/autodiff tests r? @ZuseZ4
r? @ZuseZ4