diff options
| author | Hongtao Yu <hoy@fb.com> | 2021-04-12 23:51:44 -0700 |
|---|---|---|
| committer | Hongtao Yu <hoy@fb.com> | 2021-04-19 17:55:35 -0700 |
| commit | b98807df05cb4cd728e744c3349a9dfd341f6f61 (patch) | |
| tree | 7288dd26f041df9f156887793a1bcb301c4c4d3f /llvm/lib/CodeGen/MachineSink.cpp | |
| parent | 1812319292e0041e7e32ad7b61d7252a450b95c2 (diff) | |
| download | llvm-b98807df05cb4cd728e744c3349a9dfd341f6f61.zip llvm-b98807df05cb4cd728e744c3349a9dfd341f6f61.tar.gz llvm-b98807df05cb4cd728e744c3349a9dfd341f6f61.tar.bz2 | |
[CSSPGO] Exclude pseudo probes from slot index
Pseudo probe are currently given a slot index like other regular instructions. This affects register pressure and lifetime weight computation because of enlarged lifetime length with pseudo probe instructions. As a consequence, program could get different code generated w/ and w/o pseudo probes. I'm closing the gap by excluding pseudo probes from stack index and downstream register allocation related passes.
Reviewed By: wmi
Differential Revision: https://reviews.llvm.org/D100334
Diffstat (limited to 'llvm/lib/CodeGen/MachineSink.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/MachineSink.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/llvm/lib/CodeGen/MachineSink.cpp b/llvm/lib/CodeGen/MachineSink.cpp index 6be9f4e..41fa3a2 100644 --- a/llvm/lib/CodeGen/MachineSink.cpp +++ b/llvm/lib/CodeGen/MachineSink.cpp @@ -530,7 +530,7 @@ bool MachineSinking::ProcessBlock(MachineBasicBlock &MBB) { if (!ProcessedBegin) --I; - if (MI.isDebugInstr()) { + if (MI.isDebugOrPseudoInstr()) { if (MI.isDebugValue()) ProcessDbgInst(MI); continue; @@ -718,7 +718,7 @@ MachineSinking::getBBRegisterPressure(MachineBasicBlock &MBB) { MIE = MBB.instr_begin(); MII != MIE; --MII) { MachineInstr &MI = *std::prev(MII); - if (MI.isDebugValue() || MI.isDebugLabel()) + if (MI.isDebugValue() || MI.isDebugLabel() || MI.isPseudoProbe()) continue; RegisterOperands RegOpers; RegOpers.collect(MI, *TRI, *MRI, false, false); @@ -1755,7 +1755,7 @@ bool PostRAMachineSinking::tryToSinkCopy(MachineBasicBlock &CurBB, continue; } - if (MI->isDebugInstr()) + if (MI->isDebugOrPseudoInstr()) continue; // Do not move any instruction across function call. |