<p dir="ltr">LLP64 is uncertain. I guess ms-compatibilities.<br>
Could you fix it?</p>
<div class="gmail_quote">2014/08/02 9:25 "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>>:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Jul 30, 2014 at 8:33 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
> On Wed, Jul 30, 2014 at 6:44 AM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>> wrote:<br>
>> Author: chapuni<br>
>> Date: Wed Jul 30 05:44:35 2014<br>
>> New Revision: 214298<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=214298&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=214298&view=rev</a><br>
>> Log:<br>
>> Suppress clang/test/Sema/struct-packed-align.c for targeting LLP64.<br>
>><br>
>> Modified:<br>
>>     cfe/trunk/test/Sema/struct-packed-align.c<br>
>><br>
>> Modified: cfe/trunk/test/Sema/struct-packed-align.c<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/struct-packed-align.c?rev=214298&r1=214297&r2=214298&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/struct-packed-align.c?rev=214298&r1=214297&r2=214298&view=diff</a><br>

>> ==============================================================================<br>
>> --- cfe/trunk/test/Sema/struct-packed-align.c (original)<br>
>> +++ cfe/trunk/test/Sema/struct-packed-align.c Wed Jul 30 05:44:35 2014<br>
>> @@ -120,6 +120,7 @@ extern int m1[sizeof(struct packed_fas2)<br>
>>  extern int m2[__alignof(struct packed_fas2) == 1 ? 1 : -1];<br>
>><br>
>>  // Attribute aligned can round down typedefs.  PR9253<br>
>> +// REQUIRES: LP64<br>
>>  typedef long long  __attribute__((aligned(1))) nt;<br>
><br>
> Why does this test require LP64? The type here is long long, which is<br>
> 64-bits everywhere, and the only other type involved in the struct is<br>
> a char.<br>
><br>
> Also, the commit log says LLP64 and the comment says LP64...<br>
<br>
Ping.<br>
<br>
~Aaron<br>
</blockquote></div>