[PATCH] D55107: Prototype BasicAA improvements for discussion on llvm-dev

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 30 01:58:15 PST 2018


lebedev.ri added inline comments.


================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:1016
+
+    if (match(Arg, m_c_ICmp(Pred, m_Specific(V), m_ConstantInt(C))) &&
+        Pred == ICmpInst::ICMP_NE && C->isZero()) {
----------------
Constants should always have already been canonicalized to the RHS


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55107/new/

https://reviews.llvm.org/D55107





More information about the llvm-commits mailing list