[llvm] r289768 - Revert part of r289765 that is not necessary

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 21:50:46 PST 2016


Author: hfinkel
Date: Wed Dec 14 23:50:45 2016
New Revision: 289768

URL: http://llvm.org/viewvc/llvm-project?rev=289768&view=rev
Log:
Revert part of r289765 that is not necessary

CS.doesNotAccessMemory(ArgNo) and CS.onlyReadsMemory(ArgNo) calls
dataOperandHasImpliedAttr, so revert this part of r289765 because
it should not be necessary.

Modified:
    llvm/trunk/lib/Analysis/AliasAnalysis.cpp

Modified: llvm/trunk/lib/Analysis/AliasAnalysis.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/AliasAnalysis.cpp?rev=289768&r1=289767&r2=289768&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/AliasAnalysis.cpp (original)
+++ llvm/trunk/lib/Analysis/AliasAnalysis.cpp Wed Dec 14 23:50:45 2016
@@ -470,9 +470,9 @@ ModRefInfo AAResults::callCapturesBefore
     // escape.
     if (isNoAlias(MemoryLocation(*CI), MemoryLocation(Object)))
       continue;
-    if (ArgNo < CS.getNumArgOperands() && CS.doesNotAccessMemory(ArgNo))
+    if (CS.doesNotAccessMemory(ArgNo))
       continue;
-    if (ArgNo < CS.getNumArgOperands() && CS.onlyReadsMemory(ArgNo)) {
+    if (CS.onlyReadsMemory(ArgNo)) {
       R = MRI_Ref;
       continue;
     }




More information about the llvm-commits mailing list