Re: [PATCH v1 9/9] aarch64: Handle alignment when it is bigger than BIGGEST_ALIGNMENT

[Richard Sandiford]

Evgeny Karpov <evgeny.kar...@microsoft.com> writes:
> In some cases, the alignment can be bigger than BIGGEST_ALIGNMENT.
> The patch handles these cases.
>
> gcc/ChangeLog:
>
>       * config/aarch64/aarch64-coff.h (ASM_OUTPUT_ALIGNED_LOCAL):
>       Change alignment.

Can you go into more detail?  What kind of testcase requires this?

Thanks,
Richard

> ---
>  gcc/config/aarch64/aarch64-coff.h | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/gcc/config/aarch64/aarch64-coff.h 
> b/gcc/config/aarch64/aarch64-coff.h
> index 77c09df82e4..131145171a0 100644
> --- a/gcc/config/aarch64/aarch64-coff.h
> +++ b/gcc/config/aarch64/aarch64-coff.h
> @@ -58,6 +58,16 @@
>    assemble_name ((FILE), (NAME)),            \
>    fprintf ((FILE), ",%u\n", (int)(ROUNDED)))
>  
> +#define ASM_OUTPUT_ALIGNED_LOCAL(FILE, NAME, SIZE, ALIGNMENT)  \
> +  { \
> +    unsigned HOST_WIDE_INT rounded = MAX ((SIZE), 1); \
> +    unsigned HOST_WIDE_INT alignment = MAX ((ALIGNMENT), BIGGEST_ALIGNMENT); 
> \
> +    rounded += (alignment / BITS_PER_UNIT) - 1; \
> +    rounded = (rounded / (alignment / BITS_PER_UNIT) \
> +      * (alignment / BITS_PER_UNIT)); \
> +    ASM_OUTPUT_LOCAL (FILE, NAME, SIZE, rounded); \
> +  }
> +
>  #define ASM_OUTPUT_SKIP(STREAM, NBYTES)      \
>    fprintf (STREAM, "\t.space\t%d  // skip\n", (int) (NBYTES))