Details
- Reviewers
mkazantsev - Commits
- rG767eb9f9d508: [Test] Add loop deletion switch tests
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/Transforms/LoopDeletion/eval_first_iteration.ll | ||
---|---|---|
1177 | If you want this test to exercise the same scenario as the other tests here, I guess you want %sub to be coming from %ontwo. In this case, %merge.phi will simplify to %sub and not to 2. And then %sum.next should simplify to sum + sub = sum + 2 - sum = 2. In your version, sum just happens to match 2, but I'm not sure if it was the intention of this test. | |
1240 | Just as above, I'd expect %sub coming from %second_block here to exercise the same scenario as other tests. |
llvm/test/Transforms/LoopDeletion/eval_first_iteration.ll | ||
---|---|---|
1190 | Please add a test where switch has 2 branches jumping directly to backedge, not to some intermediate block. |
Now %merge.phi will always equal to %sub instead of a constant in first two test cases.
Added a test in which two switch cases jump to the backedge directly
llvm/test/Transforms/LoopDeletion/eval_first_iteration.ll | ||
---|---|---|
1150 | There is no such block in these tests. Did you forget to re-run update_tests.py? |
There is no such block in these tests. Did you forget to re-run update_tests.py?