Skip to content

[AMDGPU] Fix 3495d04 MIR test #114963

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 5, 2024
Merged

[AMDGPU] Fix 3495d04 MIR test #114963

merged 1 commit into from
Nov 5, 2024

Conversation

optimisan
Copy link
Contributor

@optimisan optimisan commented Nov 5, 2024

Needed to specify scratchRSrcReg and spreg in order to stop after prologepilog.

Needed to specify scratchRSrcReg and spreg in order to stop after
prologepilog
@llvmbot
Copy link
Member

llvmbot commented Nov 5, 2024

@llvm/pr-subscribers-backend-amdgpu

Author: Akshat Oke (optimisan)

Changes

Needed to specify scratchRSrcReg and spreg in order to stop after prologepilog.


Full diff: https://github.com/llvm/llvm-project/pull/114963.diff

1 Files Affected:

  • (modified) llvm/test/CodeGen/MIR/AMDGPU/spill-phys-vgprs.mir (+5)
diff --git a/llvm/test/CodeGen/MIR/AMDGPU/spill-phys-vgprs.mir b/llvm/test/CodeGen/MIR/AMDGPU/spill-phys-vgprs.mir
index 3961330070c779..4d6e33cf0b68aa 100644
--- a/llvm/test/CodeGen/MIR/AMDGPU/spill-phys-vgprs.mir
+++ b/llvm/test/CodeGen/MIR/AMDGPU/spill-phys-vgprs.mir
@@ -6,6 +6,11 @@
 ---
 name: csr_sgpr_spill
 tracksRegLiveness: true
+machineFunctionInfo:
+  isEntryFunction: false
+  scratchRSrcReg: '$sgpr0_sgpr1_sgpr2_sgpr3'
+  stackPtrOffsetReg: '$sgpr32'
+  hasSpilledSGPRs: true
 body: |
   bb.0:
     S_NOP 0

@optimisan optimisan requested a review from cdevadas November 5, 2024 10:04
@optimisan optimisan merged commit e76d921 into llvm:main Nov 5, 2024
7 of 9 checks passed
PhilippRados pushed a commit to PhilippRados/llvm-project that referenced this pull request Nov 6, 2024
Needed to specify scratchRSrcReg and spreg in order to stop after
prologepilog.

- Fixes llvm#113129 test failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants