[all-commits] [llvm/llvm-project] f104eb: [AMDGPU] Reintroduce CC exception for non-inlined ...

Pierre van Houtryve via All-commits all-commits at lists.llvm.org
Tue May 23 00:01:53 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f104eb6e15503b770734e3a59937c9df865b2814
      https://github.com/llvm/llvm-project/commit/f104eb6e15503b770734e3a59937c9df865b2814
  Author: pvanhout <pierre.vanhoutryve at amd.com>
  Date:   2023-05-23 (Tue, 23 May 2023)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPUPromoteAlloca.cpp
    M llvm/test/CodeGen/AMDGPU/vector-alloca-limits.ll

  Log Message:
  -----------
  [AMDGPU] Reintroduce CC exception for non-inlined functions in Promote Alloca limits

This is basically a partial revert of https://reviews.llvm.org/D145586 ( fd1d60873fdc )

D145586 was originally introduced to help with SWDEV-363662, and it did, but
it also caused a 25% drop in performance in
some MIOpen benchmarks where, it seems,
functions are inlined more conservatively.

This patch restores the pre-D145586 behavior
for PromoteAlloca: functions with a non-entry CC
have a 32 VGPRs threshold, but only if the function
is not marked with "alwaysinline".

A good number of AMDGPU code makes uses of
the AMDGPUAlwaysInline pass anyway, so in our
backend "alwaysinline" seems very common.

This change does not affect SWDEV-363662 (the motivating issue for introducing D145586).

Fixes SWDEV-399519

Reviewed By: rampitec, #amdgpu

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




More information about the All-commits mailing list