[PATCH] D16225: [BasicAliasAnalysis] Take into account operand bundles in the getModRefInfo function

[Igor Laevsky via llvm-commits]

igor-laevsky marked an inline comment as done.

================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:734
@@ -733,3 +733,3 @@
       // escape.
       AliasResult AR =
           getBestAAResults().alias(MemoryLocation(*CI), MemoryLocation(Object));
----------------
reames wrote:
> A potential enhancement to this code would be to return MRI_Ref if the arguments which could alias the object are all readonly.  This would be potentially useful for deopt bundles since all the arguments are implicitly read only.  
> 
> If you decide to do this, it should definitely be a separate patch.  
That makes sense. I'll send a separate patch.


Repository:
  rL LLVM

http://reviews.llvm.org/D16225