[PATCH] D13992: [AliasAnalysis] Take into account readonly attribute for the function arguments

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 12:58:25 PDT 2015


hfinkel added a subscriber: hfinkel.

================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:619
@@ -618,1 +618,3 @@
 
+  if (CS.paramHasAttr(ArgIdx + 1, Attribute::ReadOnly))
+    return MRI_Ref;
----------------
Don't you need to check all arguments, not just the first? (please add some corresponding test cases)



http://reviews.llvm.org/D13992





More information about the llvm-commits mailing list