[PATCH] D25913: [InstCombine] Fold nuw left-shifts in `ugt`/`ule` comparisons.
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 26 13:28:23 PDT 2016
spatel added reviewers: majnemer, efriedma.
spatel added a comment.
Yes, adding the derivation to the comments would be good.
I wasn't really concerned about the speed of the extra check for *C != 0; it just simplified the code to not have the extra check for that half of the transform. Since I've gotten that sort of assumption wrong before, I'm cc'ing David and Eli to see if they have a better idea.
More information about the llvm-commits