On Aug 11, 2016, at 12:40 AM, Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> wrote: > > 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?
LGTM. Ok. If anyone else has a reason why it would be bad, don't let me discourage you from speaking up. > +/* { dg-skip-if "No alignment restrictions" { { ! natural_alignment_32 } && > { ! natural_alignment_64 } } } */