On 08/16/2016 11:55 PM, Senthil Kumar Selvaraj wrote:
Jeff Law writes:
On 08/11/2016 01:40 AM, Senthil Kumar Selvaraj wrote:
Hi,
The below patch adds the AVR target to the list of targets that don't
have natural_alignment_32. It also skips ipa/propalign-*.c
tests (which expect 4 byte alignment), if both
natural_alignment_32 and natural_alignment_64 are false.
Is this the right way to fix this? Ok to commit?
Regards
Senthil
gcc/testsuite/ChangeLog:
2016-08-11 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com>
* gcc.dg/ipa/propalign-1.c: Skip for targets with !natural_alignment_32
and !natural_alignment_64.
* gcc.dg/ipa/propalign-2.c: Likewise.
* gcc.dg/ipa/propalign-3.c: Likewise.
* gcc.dg/ipa/propalign-4.c: Likewise.
* gcc.dg/ipa/propalign-5.c: Likewise.
* lib/target-supports.exp
(check_effective_target_natural_alignment_32): Add avr-*-*.
Do you need to add an avr case to
check_effective_target_natural_alignment_64 as well?
The 64 bit version is written to return true only for lp64 and spu-*-*,
so I didn't have to do anything there.
Have you tested this on anything other than avr?
Yes, x86_64-pc-linux passes these tests, like it did before my patch.
OK for the trunk. Please install.
Thanks,
jeff
