[llvm] [DAG] Fold urem(urem(A, BCst), Op1Cst) -> urem(A, Op1Cst) (PR #159517)

via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 18 02:15:23 PDT 2025


kper wrote:

> The logic looks correct to me. But should it have a "one use" check, like was mentioned in #157644?

I wasn't really sure. We only added it because it showed that we're breaking up the div+rem patterns. Beside that there wasn't a functional reason. So should I add it again?

https://github.com/llvm/llvm-project/pull/159517


More information about the llvm-commits mailing list