[dragonegg] r174494 - It is normal that this test fails on x86-32. What is unclear is why it

Bill Wendling wendling at apple.com
Wed Feb 6 11:52:09 PST 2013


On Feb 6, 2013, at 12:41 AM, Duncan Sands <baldrick at free.fr> wrote:

> Hi Bill, it looks like you changed the test in commit 174125:
> 
>  "Update test so that it gets the attribute before 'byval'"
> 
> -// CHECK: %struct.T* byval
> +// CHECK: %struct.T* align 8 byval
> 
> The new check will only pass on x86-64.  This is OK, as the original problem was
> on x86-64 only.

Ah! Okay then. :)

> This commit didn't turn up in the buildbot blame list for some reason.
> 
> Anyway, everything is clear to me now, and there is nothing you need to worry
> about!  I appreciate your trying to keep dragonegg chugging along :)
> 
No problem. :)

-bw




More information about the llvm-commits mailing list