On Thu, Jul 9, 2015 at 12:53 AM, H.J. Lu <hongjiu...@intel.com> wrote: > attribute ((aligned)) should align to the minimum of BIGGEST_ALIGNMENT, > which is 4 bytes for -miamcu. > > Tested on Linux/x86-64. OK for trunk? > > Thanks. > > H.J. > --- > gcc/ > > PR target/66818 > * config/i386/i386.h (ATTRIBUTE_ALIGNED_VALUE): Defined to 32 > for IA MCU. > > gcc/testsuite/ > > PR target/66818 > * gcc.target/i386/pr66818.c: New test.
OK. Thanks, Uros. > --- > gcc/config/i386/i386.h | 2 +- > gcc/testsuite/gcc.target/i386/pr66818.c | 5 +++++ > 2 files changed, 6 insertions(+), 1 deletion(-) > create mode 100644 gcc/testsuite/gcc.target/i386/pr66818.c > > diff --git a/gcc/config/i386/i386.h b/gcc/config/i386/i386.h > index f357e79..74334ff 100644 > --- a/gcc/config/i386/i386.h > +++ b/gcc/config/i386/i386.h > @@ -811,7 +811,7 @@ extern const char *host_detect_local_cpu (int argc, const > char **argv); > > /* Alignment value for attribute ((aligned)). It is a constant since > it is the part of the ABI. We shouldn't change it with -mavx. */ > -#define ATTRIBUTE_ALIGNED_VALUE 128 > +#define ATTRIBUTE_ALIGNED_VALUE (TARGET_IAMCU ? 32 : 128) > > /* Decide whether a variable of mode MODE should be 128 bit aligned. */ > #define ALIGN_MODE_128(MODE) \ > diff --git a/gcc/testsuite/gcc.target/i386/pr66818.c > b/gcc/testsuite/gcc.target/i386/pr66818.c > new file mode 100644 > index 0000000..d90394c > --- /dev/null > +++ b/gcc/testsuite/gcc.target/i386/pr66818.c > @@ -0,0 +1,5 @@ > +/* { dg-do compile { target ia32 } } */ > +/* { dg-options "-O2 -mno-sse -mno-mmx -miamcu" } */ > + > +struct dummy { int x __attribute__((aligned)); }; > +int array[__alignof__(struct dummy) == 4 ? 1 : -1]; > -- > 2.4.3 >