[llvm-commits] [PATCH] Teach MachineInstr::isRegTiedToDefOperand() to correctly parse inline asm operands

[Evan Cheng]

Sure. Thanks.

Evan
On Jul 16, 2009, at 8:15 AM, Jakob Stoklund Olesen wrote:

> Hi,
>
> The method MachineInstr:: isRegTiedToDefOperand() tries to find the  
> flag operand on an inline asm instruction by going backwards from a  
> register operand until it sees an immediate operand. This fails for  
> PowerPC "m"-type asm operands represented as (flag, imm, reg)  
> triples, causing assertions when imm<0.
>
> The attached patch changes isRegTiedToDefOperand to search for the  
> operand flag from the beginning of the operand list. It is the only  
> safe way.
>
> After applying this patch the the machine code verifier no longer  
> trips assertions in the CodeGen/PowerPC test suite.
>
> Regards,
> /jakob
> <asm-operands.patch>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits