[llvm-commits] [llvm] r119922 - in /llvm/trunk: lib/Target/README.txt lib/Transforms/InstCombine/InstCombineCompares.cpp test/Transforms/InstCombine/icmp.ll

Duncan Sands baldrick at free.fr
Mon Nov 22 00:23:36 PST 2010


Hi Chris,

> Thanks for the review!

now I think about it some more, aren't all the conditions inverted?
Consider this:

         // If the LHS is 1 << x, and we know the result is a power of 2 like 8,
         // then turn "((1 << x)&8) == 0" into "x == 3".

Shouldn't that be "x != 3" not "x == 3"?  Likewise for the others.

Ciao,

Duncan.



More information about the llvm-commits mailing list