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 } } } */

Reply via email to