[PATCH] D84617: [ValueTracking] Remove AllocaForValue parameter

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 26 22:37:18 PDT 2020


vitalybuka created this revision.
vitalybuka added reviewers: eugenis, glider.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

findAllocaForValue uses AllocaForValue to cache resolved values.
The function is used only to resolve arguments of lifetime
intrinsic which usually are not fare for allocas. So result reuse
is likely unnoticeable.

In followup patches I'd like to replace the function with
GetUnderlyingObjects.

Depends on D84616 <https://reviews.llvm.org/D84616>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84617

Files:
  llvm/include/llvm/Analysis/ValueTracking.h
  llvm/lib/Analysis/ValueTracking.cpp
  llvm/lib/Target/AArch64/AArch64StackTagging.cpp
  llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
  llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84617.280788.patch
Type: text/x-patch
Size: 4377 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200727/48a96d13/attachment.bin>


More information about the llvm-commits mailing list