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?