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?