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

[Renato Golin]

Hi Vinicius,

Is that without any extra patch? Or do you apply the old Weimin's patch
that you were before?

Does "it works again" mean it boots? Is this just the kernel or Android user

cheers,
--renato
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/20130712/05e3a842/attachment.html>