[PATCH] D48066: Add one more No-alias case to alias analysis.

[JinGu Kang via Phabricator via llvm-commits]

jaykang10 added inline comments.


================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:1813
+    if (ConstantInt *LHSC = dyn_cast<ConstantInt>(BOp->getOperand(0))) {
+      APInt LHS = LHSC->getValue().zextOrSelf(BitWidth);
+      // Check odd number.
----------------
nlopes wrote:
> you don't need the zext here. The value should have at least 1 bit, so that's sufficient to test for even/odd.
Oops, sorry... I have updated it with @hfinkel 's comment.


https://reviews.llvm.org/D48066