[all-commits] [llvm/llvm-project] e3515b: Reapply "[AMDGPU] Modify adjustInliningThreshold t...

Janek van Oirschot via All-commits all-commits at lists.llvm.org
Mon Feb 13 04:17:57 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e3515ba3816b9cabeca6a3b03b90902ebcfd3c65
      https://github.com/llvm/llvm-project/commit/e3515ba3816b9cabeca6a3b03b90902ebcfd3c65
  Author: Janek van Oirschot <janek.vanoirschot at amd.com>
  Date:   2023-02-13 (Mon, 13 Feb 2023)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
    M llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
    M llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-argument-i64.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-argument.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-array-ptr-argument.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-indirect-call-argument.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-ptr-argument.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-struct-argument.ll
    A llvm/test/Transforms/Inline/AMDGPU/amdgpu-inline-stack-vector-ptr-argument.ll

  Log Message:
  -----------
  Reapply "[AMDGPU] Modify adjustInliningThreshold to also consider the cost of passing function arguments through the stack"

Reapplies 142c28ffa1323e9a8d53200a22c80d5d778e0d0f as part of D140242 which got reverted due to amdgpu openmp test failures.

This diff fixes said failures by eliding most of `adjustInliningThresholdUsingCallee` for indirect calls as the callee function is unavailable for indirect calls.

Reviewed By: arsenm, #amdgpu

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




More information about the All-commits mailing list