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

[Weiming Zhao]

Hi Jakob,

Attached is my 2nd patch that replaces hard coded [R0, R1] of
Intrinsic::arm_ldrexd and [R2, R3] of Intrinsic::arm_strexd 
I plan to include the inlineASM fix into next patch. 

This patch passes the unit tests especially test/CodeGen/ARM/ldstrexd.ll

Please help to review it.

Thanks,
Weiming

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

-----Original Message-----
From: Jakob Stoklund Olesen [mailto:stoklund at 2pi.dk] 
Sent: Thursday, October 25, 2012 9:31 AM
To: weimingz at codeaurora.org
Cc: 'Jim Grosbach'; llvm-commits at cs.uiuc.edu; zinob at codeaurora.org
Subject: Re: [llvm-commits] Fixing Bug 13662: paired register for inline asm
with 64-bit data on ARM


On Oct 23, 2012, at 2:57 PM, Weiming Zhao <weimingz at codeaurora.org> wrote:

> When I run "make check", even without my patch, I still see 93 
> unexpected failures. So maybe I ignored that failure.

Please get that fixed before sending more patches. I can't do basic checking
for you.

/jakob

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Remove-hard-coded-registers-in-ARM-ldrexd-and-strexd.patch
Type: application/octet-stream
Size: 5912 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121026/2bd52feb/attachment.obj>