[PATCH] D28394: [AliasAnalysis] Limit `MemoryLocation` retrieval to valid intrinsic arguments.

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 6 12:08:17 PST 2017


reames added inline comments.


================
Comment at: lib/Analysis/AliasAnalysis.cpp:177
+        unsigned ArgIdx = A.Index;
+        if (!Arg->getType()->isPointerTy() || CS.doesNotAccessMemory(ArgIdx))
           continue;
----------------
I'm missing something here.  *Why* is it problematic to ask for a memory location of the parameter to invariant_end?  I would expect that to just work.  


Repository:
  rL LLVM

https://reviews.llvm.org/D28394





More information about the llvm-commits mailing list