Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| llvm/test/Transforms/InstCombine/urem-mul.ll | ||
|---|---|---|
| 33 | 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? | |
| 59 | This could be called something like urem_XY_XZ_constY_is_lt_constZ to reflect why we return BO0 | |
| 150 | %X is unused | |
| llvm/test/Transforms/InstCombine/urem-mul.ll | ||
|---|---|---|
| 33 |
| |
| 33 | 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?