Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| llvm/test/Transforms/InstCombine/urem-mul.ll | ||
|---|---|---|
| 32 | This test could be better named, since C is actually %X. The important thing here which lets us simplify this to 0 is that the constant LHS multiplicand >= the constant RHS multiplicand of the rem. Maybe something like urem_XY_XZ_constY_is_gte_constZ? | |
| 58 | This could be called something like urem_XY_XZ_constY_is_lt_constZ to reflect why we return BO0 | |
| 149 | %X is unused | |
| llvm/test/Transforms/InstCombine/urem-mul.ll | ||
|---|---|---|
| 32 |
| |
| 32 | Okay, renamed all functions to reflect the conditions rather than the result. | |
This test could be better named, since C is actually %X. The important thing here which lets us simplify this to 0 is that the constant LHS multiplicand >= the constant RHS multiplicand of the rem. Maybe something like urem_XY_XZ_constY_is_gte_constZ?