[all-commits] [llvm/llvm-project] 735a5f: [AMDGPU] When allocating VGPRs, VGPR spills are no...

Jay Foad via All-commits all-commits at lists.llvm.org
Mon Sep 30 05:25:17 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 735a5f67e351fc072a9f7c18b030036681f7935a
      https://github.com/llvm/llvm-project/commit/735a5f67e351fc072a9f7c18b030036681f7935a
  Author: Jay Foad <jay.foad at amd.com>
  Date:   2024-09-30 (Mon, 30 Sep 2024)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIInstrInfo.cpp

  Log Message:
  -----------
  [AMDGPU] When allocating VGPRs, VGPR spills are not part of the prologue (#109439)

PRs #69924 and #72140 modified SIInstrInfo::isBasicBlockPrologue to skip
over EXEC modifications and spills when allocating VGPRs. But treating
VGPR spills as part of the prologue can confuse the register allocator
as in #109294, so restrict it to SGPR spills, which were inserted during
SGPR allocation which is done in an earlier pass.

Fixes: #109294
Fixes: SWDEV-485841



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list