[PATCH] D142542: [InstSimplify] Simplify icmp between Shl instructions of the same value
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 15 00:10:18 PST 2023
nikic added a reviewer: nikic.
nikic added a comment.
Can you please include the relevant alive2 proofs for the transform in the patch description? Should be something along the lines of https://alive2.llvm.org/ce/z/KCrE8j and https://alive2.llvm.org/ce/z/wH8vBN I think. Notably, a signed predicate requires *both* nuw and nsw. I don't think this is correctly enforced right now?
Can you please also pre-commit the tests?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142542/new/
https://reviews.llvm.org/D142542
More information about the llvm-commits
mailing list