Create a test case to test the fold for the expression pattern
'usub_sat(X, C) pred C2'.
Details
Details
- Reviewers
nikic goldstein.w.n - Commits
- rG0e71bb33f827: [InstCombine] Test cases for D154206
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
llvm/test/Transforms/InstCombine/icmp-usub-sat.ll | ||
---|---|---|
53 | ult 1 is eq 0 which is a special case. I'd adjust the constant to be 2 here. | |
178 | Pick different constants so it's not always false? | |
257 | The standard pattern for multi-use tests is call void @use(i8 %sub). Avoids any funny folds. | |
276 | As you don't allow multi-use folds, it's enough to have a single test to cover this -- there's no much in point in having 10 tests that all show the same thing. |
Comment Actions
- Address reviewer comments
- Fix cases which return true/false/zeroinitializer
- Fix (and remove a lot of) multi-use tests
- Reorganize some cases
ult 1 is eq 0 which is a special case. I'd adjust the constant to be 2 here.