[PATCH] MachineVerifier: Don't crash if MachineOperand has no parent

Matt Arsenault arsenm2 at gmail.com
Wed Apr 29 16:01:24 PDT 2015


> On Apr 29, 2015, at 3:45 PM, Quentin Colombet <qcolombet at apple.com> wrote:
> 
> Hmm, in that case, why do we need that check?
> 
> I just wonder how we could end up in such situation if we use the dedicated APIs.

What I was trying to do when I ran into this was:

There is no set* or ChangeTo* function for the MachineOperand type I wanted to change to, so I did basically

MachineOperand &Op = MI->getOperand(I)
Op = MachineOperand::CreateES(“foo”);
// MI now has operand Op which doesn’t have the parent set


This worked, except would crash the verifier

-Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150429/24f5e726/attachment.html>


More information about the llvm-commits mailing list