[PATCH] PR16490: Fix a crash in ARM inline asm

Tinti viniciustinti at gmail.com
Thu Jul 11 10:25:04 PDT 2013


Hi all,

Looks like if you revert the following commits it works again.

8b97c23fe95db0bba23855205e2318188ceea947 (HEAD, test) Revert "Bug 13662:
Enable GPRPair for all i64 operands of inline asm on ARM"
a189501caeea75082f70646afd449700bdd67ad3 Revert "Remove unused variables."
20f544be14e5096ee59c5c383731fb425191d5e7 Revert "PR16490: fix a crash in
ARMDAGToDAGISel::SelectInlineAsm."
5762e77973991da82bc0fd3947df1a5d590de12e Revert "Add a comment to this
change, requested by Eric Christopher."

On top of these:
12fa02841ca1f2115d38aadd3e5d7afe08c65337 (origin/master, origin/HEAD,
master) Merge these tests.
aad52dfef9b64ed00aed3626513f3d2eea0fb8e0 Use a more unique name to avoid
conflicting with directory.ll tests when running in parallel.

Best regards,
Tinti


On Tue, Jul 9, 2013 at 6:06 PM, Jan-Simon Möller <dl9pf at gmx.de> wrote:

> Hi all,
>
> On Monday 08 July 2013 14:09:07 Joey Gouly wrote:
> > So, I think my patch is actually orthogonal to r185169. It’s just that
> > r185169 exposed a latent bug, that I fixed with my commit.
> >
> > I can't think of a good way to find the problem here, apart from diffing
> the
> > kernels before r185169 and at r185169 (with my patch to let it compile).
> > Seeing where the differences come from, and finding the inline assembly
> > that produced that.
>
> @Joe - the output is here:
>
> http://88.198.106.157/tmp_vmlinux1_old.xz   (pre r185169)
> http://88.198.106.157/tmp_vmlinux1_new.xz   (r185169+your commit)
>
> What are the next steps to compare them ? What do you propose ?
>
> -- JS
>
>


-- 
Simplicity is the ultimate sophistication
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130711/a449e88f/attachment.html>


More information about the llvm-commits mailing list