[all-commits] [llvm/llvm-project] a09f79: TargetTransformInfo: Add addrspacesMayAlias

Matt Arsenault via All-commits all-commits at lists.llvm.org
Tue Jun 13 17:44:22 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a09f79d2275f29b927b6f9c0472d49f6f1bfd029
      https://github.com/llvm/llvm-project/commit/a09f79d2275f29b927b6f9c0472d49f6f1bfd029
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2023-06-13 (Tue, 13 Jun 2023)

  Changed paths:
    M llvm/include/llvm/Analysis/TargetTransformInfo.h
    M llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/lib/Analysis/TargetTransformInfo.cpp
    M llvm/lib/Target/AMDGPU/AMDGPU.h
    M llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.h

  Log Message:
  -----------
  TargetTransformInfo: Add addrspacesMayAlias

For some reason we used to only handle address space aliasing through
chaining a target specific AA pass. We need never-fail simple queries
in order to lower memmove intrinsics based purely on the address
spaces.

I also think it would be better if BasicAA checked this, rather than
relying on the target AA passes. Currently we go through the more
expensive AA analyses before getting to the trivial address space
checks.




More information about the All-commits mailing list