[llvm-commits] [PATCH] PR14992 - Tablegen incorrectly converts ARM tLDMIA_UPD pseudo to tLDMIA

[David Peixotto]

Thanks for taking a look, Tim. I've attached an updated patch with the test
case from the bug report.

-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted
by The Linux Foundation



> -----Original Message-----
> From: Tim Northover [mailto:t.p.northover at gmail.com]
> Sent: Saturday, January 19, 2013 12:44 AM
> To: David Peixotto
> Cc: llvm-commits
> Subject: Re: [llvm-commits] [PATCH] PR14992 - Tablegen incorrectly
converts
> ARM tLDMIA_UPD pseudo to tLDMIA
> 
> > I'd like a tablegen expert to take a look. I tried the patch and saw
> > only one change (the change that fixes the bug) in the .inc files for
> > the x86 and arm backends.
> 
> The change itself looks reasonable to me: assuming the variable_ops parts
of
> each Instruction have the same purpose is the only solution that could
> generically work. (Actually, the reverse of this situation, DestOps >
SrcOps,
> would be even worse and could lead to reading non-existent operands from
> Src).
> 
> Could you include the test you given though?
> 
> Cheers.
> 
> Tim.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-PR14992-Tablegen-incorrectly-converts-ARM-tLDMIA_UPD.patch
Type: application/octet-stream
Size: 2553 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130121/a98c7e34/attachment.obj>