[PATCH] D13074: [InstSimplify] Fold simple known implications to true
hfinkel@anl.gov via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 23 13:25:04 PDT 2015
hfinkel added a subscriber: hfinkel.
================
Comment at: lib/Analysis/InstructionSimplify.cpp:2216
@@ -2174,3 +2215,3 @@
break;
case ICmpInst::ICMP_UGE:
// X >=u 1 -> X
----------------
reames wrote:
> Pretty sure I can't use signed compares. I believe "1" is interpreted as "-1" in a signed compare. This generates a different truth table.
FWIW, yes, i1 signed values are interpreted as 0 and -1.
http://reviews.llvm.org/D13074
More information about the llvm-commits
mailing list