[llvm] [ANDGPU][NFC] Remove FIXME that has been resolved (PR #79047)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 12:12:44 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-amdgpu

Author: AtariDreams (AtariDreams)

<details>
<summary>Changes</summary>

As of 5aea839ab3edeb58bb0eea833ad9427d0ae7ef67, this FIXME has been resolved.

So we can remove this comment.

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


1 Files Affected:

- (modified) llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp (-1) 


``````````diff
diff --git a/llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp b/llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
index a2cacb5cbaa393a..8f9d95c6d4b1421 100644
--- a/llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
@@ -1619,7 +1619,6 @@ void SIRegisterInfo::buildSpillLoadStore(
       }
       SubReg = TmpIntermediateVGPR;
     } else if (UseVGPROffset) {
-      // FIXME: change to scavengeRegisterBackwards()
       if (!TmpOffsetVGPR) {
         TmpOffsetVGPR = RS->scavengeRegisterBackwards(AMDGPU::VGPR_32RegClass,
                                                       MI, false, 0);

``````````

</details>


https://github.com/llvm/llvm-project/pull/79047


More information about the llvm-commits mailing list