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

Evan Cheng evan.cheng at apple.com
Thu Nov 29 11:57:51 PST 2012


LGTM
On Nov 28, 2012, at 9:50 AM, Weiming Zhao <weimingz at codeaurora.org> wrote:

> Ping …
>  
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
>  
> From: Weiming Zhao [mailto:weimingz at codeaurora.org] 
> Sent: Monday, November 19, 2012 3:13 PM
> To: 'Jakob Stoklund Olesen'; 'llvm-commits at cs.uiuc.edu'; 'grosbach at apple.com'
> Subject: Fixing Bug 13662: paired register for inline asm with 64-bit data on ARM
>  
> Hi Jakob & Jim,
>  
> Sorry for keep you busy, but here is the last part of my patches. :D
> It enables inline asm of ldrexd/strexd to work correctly.
>  
> I have to change CodeGen/SelectionDAG/InstrEmitter.cpp a litte bit because: for the inline asm nodes, I have to put a virtual reg instead of a value node.
> And due to “untyped” type, I can’t insert CopyFromReg or CopyToReg as the framework won’t find the GPRPairReg class.
> So I have to directly connect ExtractSubReg, RegSequence with a Register SDNode.
>  
> Please help to review it.
>  
> Thanks,
> Weiming
>  
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
>  
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121129/5f8f5963/attachment.html>


More information about the llvm-commits mailing list