On Wed, Jul 30, 2014 at 6:44 AM, NAKAMURA Takumi <[email protected]> wrote: > Author: chapuni > Date: Wed Jul 30 05:44:35 2014 > New Revision: 214298 > > URL: http://llvm.org/viewvc/llvm-project?rev=214298&view=rev > Log: > Suppress clang/test/Sema/struct-packed-align.c for targeting LLP64. > > Modified: > cfe/trunk/test/Sema/struct-packed-align.c > > Modified: cfe/trunk/test/Sema/struct-packed-align.c > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/struct-packed-align.c?rev=214298&r1=214297&r2=214298&view=diff > ============================================================================== > --- cfe/trunk/test/Sema/struct-packed-align.c (original) > +++ cfe/trunk/test/Sema/struct-packed-align.c Wed Jul 30 05:44:35 2014 > @@ -120,6 +120,7 @@ extern int m1[sizeof(struct packed_fas2) > extern int m2[__alignof(struct packed_fas2) == 1 ? 1 : -1]; > > // Attribute aligned can round down typedefs. PR9253 > +// REQUIRES: LP64 > typedef long long __attribute__((aligned(1))) nt;
Why does this test require LP64? The type here is long long, which is 64-bits everywhere, and the only other type involved in the struct is a char. Also, the commit log says LLP64 and the comment says LP64... ~Aaron _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
