Skip to content

Commit

Permalink
[AMDGPU] Remove outdated FIXME in comments [NFC]
Browse files Browse the repository at this point in the history
This case has already been handled by D106449.
  • Loading branch information
piggynl committed Mar 2, 2023
1 parent c6b9df0 commit 8fccdfa
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
Expand Up @@ -2555,12 +2555,6 @@ void SIInstrInfo::insertIndirectBranch(MachineBasicBlock &MBB,
BuildMI(&MBB, DL, get(AMDGPU::S_SETPC_B64))
.addReg(PCReg);

// FIXME: If spilling is necessary, this will fail because this scavenger has
// no emergency stack slots. It is non-trivial to spill in this situation,
// because the restore code needs to be specially placed after the
// jump. BranchRelaxation then needs to be made aware of the newly inserted
// block.
//
// If a spill is needed for the pc register pair, we need to insert a spill
// restore block right before the destination block, and insert a short branch
// into the old destination block's fallthrough predecessor.
Expand Down

0 comments on commit 8fccdfa

Please sign in to comment.