This was creating a copy of the register the pseudo itself was
def'ing, leaving a copy of an undefined register. I'm not sure how
the verifier is not catching this, but this avoids asserting in a
future change to RegAllocFast
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
The patch LGTM. SimonD, as far as I know you are an author of original patch D31287. Do you have any comments / objections against this fix?
Comment Actions
Good catch. LGTM.
lib/Target/Mips/MipsISelLowering.cpp | ||
---|---|---|
1727 | This can committed separately as it is a NFC. |
This can committed separately as it is a NFC.