[llvm-commits] Fixing Bug 13662: paired register for inline asm with 64-bit data on ARM

Jakob Stoklund Olesen stoklund at 2pi.dk
Wed Feb 6 16:24:28 PST 2013


On Jan 29, 2013, at 3:01 PM, Weiming Zhao <weimingz at codeaurora.org> wrote:

> Hi Jakob,
>  
> Attached is the last piece of the whole patch.
> It adds ARM specific support of inline asm of ldrexd/strexd
>  
> Please help to review it.

This basically looks good, but you need multiple test cases.

I would prefer to handle this kind of constraint in target independent code, otherwise the next target will also need to add a big blob of inline asm special case code. I think it is outside the scope of this patch, though.

/jakob

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130206/3c13e8a2/attachment.html>


More information about the llvm-commits mailing list