[PATCH] D24700: [InstCombine] optimize unsigned icmp of increment

[Sanjay Patel via llvm-commits]

spatel added inline comments.

================
Comment at: test/Transforms/InstCombine/icmp.ll:2799
@@ +2798,3 @@
+; CHECK-LABEL: @cmp_ult_rhs_inc(
+; CHECK-NEXT:    [[CONV:%.*]] = fptosi float %x to i32
+; CHECK-NEXT:    [[CMP:%.*]] = icmp ule i32 [[CONV]], %i
----------------
The commuting happens because of operand complexity canonicalization. It would be nice to add a comment above here to make that clear.

Please see:
https://llvm.org/bugs/show_bug.cgi?id=28296 
https://reviews.llvm.org/D24419
...for more info. 


https://reviews.llvm.org/D24700