[PATCH] D17279: [Sink] Don't move calls to readonly functions across stores

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue May 31 08:03:21 PDT 2016


dberlin accepted this revision.
dberlin added a reviewer: dberlin.
dberlin added a comment.
This revision is now accepted and ready to land.

Please add a test for the getmodref info change and commit it separately.


================
Comment at: lib/Analysis/AliasAnalysis.cpp:112
@@ -111,3 +111,3 @@
     // Check if the two calls modify the same memory
-    return getModRefInfo(Call, CS);
+    return getModRefInfo(CS, Call);
   } else {
----------------
Please commit this separately.


http://reviews.llvm.org/D17279





More information about the llvm-commits mailing list