[llvm] [InstCombine] Fold `(icmp pred (trunc nuw/nsw X), C)` -> `(icmp pred X, (zext/sext C))` (PR #87935)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 10:21:34 PDT 2024


nikic wrote:

I think it's okay to revert in the meantime. Though you might want to check whether https://github.com/llvm/llvm-project/pull/93163 has fixed the msan issue -- I feel like it may be related (incorrect inference of trunc nuw/nsw for values that may be undef).

> Also if we wait, it can be branched into clang RC, which will need cherry-picks, potentially with conflicts.

This is still two months away, so that's unlikely :)

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


More information about the llvm-commits mailing list