[PATCH] D50730: [AST] Generalize argument specific aliasing

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 17 15:08:47 PDT 2018


reames added inline comments.


================
Comment at: include/llvm/Analysis/MemoryLocation.h:113
+                                       const TargetLibraryInfo *TLI);
+  static MemoryLocation getForArgument(ImmutableCallSite CS, unsigned ArgIdx,
+                                       const TargetLibraryInfo &TLI) {
----------------
hiraditya wrote:
> Maybe just remove this function?
I would prefer not to.  There are a bunch of callers and I'd like to eventually pipe TLI though here as well.  I consider this a temporary step.


Repository:
  rL LLVM

https://reviews.llvm.org/D50730





More information about the llvm-commits mailing list