[PATCH] D67411: [InstSimplify] simplifyUnsignedRangeCheck(): handle more cases (PR43251)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 08:11:37 PDT 2019


spatel added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:1774-1775
   if (ICmp0 && ICmp1)
-    V = IsAnd ? simplifyAndOfICmps(ICmp0, ICmp1, Q.IIQ, Q.DL)
-              : simplifyOrOfICmps(ICmp0, ICmp1, Q.IIQ, Q.DL);
+    V = IsAnd ? simplifyAndOfICmps(ICmp0, ICmp1, Q)
+              : simplifyOrOfICmps(ICmp0, ICmp1, Q);
 
----------------
Can the diffs to thread the full SimplifyQuery through the calls be done as a preliminary step?
I'm not sure how to expose a diff in a test, but I'd think it's not quite 'NFC', so it should be an independent change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67411





More information about the llvm-commits mailing list