[PATCH] D125657: [RegisterCoalescer] fix dst subreg replacement during remat copy trick
Afanasyev Ivan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 16 21:00:33 PDT 2022
ivafanas updated this revision to Diff 429928.
ivafanas added a comment.
Typo fix in comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125657/new/
https://reviews.llvm.org/D125657
Files:
llvm/lib/CodeGen/RegisterCoalescer.cpp
Index: llvm/lib/CodeGen/RegisterCoalescer.cpp
===================================================================
--- llvm/lib/CodeGen/RegisterCoalescer.cpp
+++ llvm/lib/CodeGen/RegisterCoalescer.cpp
@@ -1379,8 +1379,19 @@
TRI->getCommonSubClass(DefRC, DstRC);
if (CommonRC != nullptr) {
NewRC = CommonRC;
+
+ // Instruction might contain "undef %0:subreg" as use operand:
+ // %0:subreg = instr op_1, ..., op_N, undef %0:subreg, op_N+2, ...
+ //
+ // Need to check all operands.
+ for (MachineOperand &MO : NewMI.operands()) {
+ if (MO.isReg() && MO.getReg() == DstReg &&
+ MO.getSubReg() == DstIdx) {
+ MO.setSubReg(0);
+ }
+ }
+
DstIdx = 0;
- DefMO.setSubReg(0);
DefMO.setIsUndef(false); // Only subregs can have def+undef.
}
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D125657.429928.patch
Type: text/x-patch
Size: 881 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220517/ea5cec50/attachment.bin>
More information about the llvm-commits
mailing list