diff options
author | Geoff Berry <gberry@codeaurora.org> | 2018-01-29 18:47:48 +0000 |
---|---|---|
committer | Geoff Berry <gberry@codeaurora.org> | 2018-01-29 18:47:48 +0000 |
commit | d37dc77b6e52d46f2388070f2fd530424b49a8cd (patch) | |
tree | fe09801732f33a3e4f05668d1eba70de4f02a05a /llvm/lib/CodeGen/MachineInstr.cpp | |
parent | d5f76ad37fb235c684d551251eb8c0e204bde134 (diff) | |
download | bcm5719-llvm-d37dc77b6e52d46f2388070f2fd530424b49a8cd.tar.gz bcm5719-llvm-d37dc77b6e52d46f2388070f2fd530424b49a8cd.zip |
[AMDGPU][X86][Mips] Make sure renamable bit not set for reserved regs
Summary:
Fix a few places that were modifying code after register
allocation to set the renamable bit correctly to avoid failing the
validation added in D42449.
llvm-svn: 323675
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
-rw-r--r-- | llvm/lib/CodeGen/MachineInstr.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/llvm/lib/CodeGen/MachineInstr.cpp b/llvm/lib/CodeGen/MachineInstr.cpp index 32785dee0cb..d2d3bc77ac4 100644 --- a/llvm/lib/CodeGen/MachineInstr.cpp +++ b/llvm/lib/CodeGen/MachineInstr.cpp @@ -928,10 +928,10 @@ void MachineInstr::clearKillInfo() { } } -void MachineInstr::substituteRegister(unsigned FromReg, - unsigned ToReg, +void MachineInstr::substituteRegister(unsigned FromReg, unsigned ToReg, unsigned SubIdx, - const TargetRegisterInfo &RegInfo) { + const TargetRegisterInfo &RegInfo, + bool ClearIsRenamable) { if (TargetRegisterInfo::isPhysicalRegister(ToReg)) { if (SubIdx) ToReg = RegInfo.getSubReg(ToReg, SubIdx); @@ -939,8 +939,11 @@ void MachineInstr::substituteRegister(unsigned FromReg, if (!MO.isReg() || MO.getReg() != FromReg) continue; MO.substPhysReg(ToReg, RegInfo); + if (ClearIsRenamable) + MO.setIsRenamable(false); } } else { + assert(!ClearIsRenamable && "IsRenamable invalid for virtual registers"); for (MachineOperand &MO : operands()) { if (!MO.isReg() || MO.getReg() != FromReg) continue; |