[all-commits] [llvm/llvm-project] b256cb: [ValueTracking] Remove AllocaForValue parameter

Vitaly Buka via All-commits all-commits at lists.llvm.org
Thu Jul 30 18:48:59 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: b256cb88a716dfd270745af1bb98b49c22521a99
      https://github.com/llvm/llvm-project/commit/b256cb88a716dfd270745af1bb98b49c22521a99
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-07-30 (Thu, 30 Jul 2020)

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

  Log Message:
  -----------
  [ValueTracking] Remove AllocaForValue parameter

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.

Differential Revision: https://reviews.llvm.org/D84617




More information about the All-commits mailing list