Re: [PATCH v2] configure: Check -falign-jumps=1 -falign-functions=1 beside -falign-loops=1

[Paul Menzel]

Dear Fāng-ruì,


Am 24.08.21 um 07:53 schrieb Fangrui Song via Grub-devel:
> Clang does not support -falign-jumps and only recently gained support
> for -falign-loops. Test all the alignment options to avoid passing
> unrecognized options to Clang:
>
>      clang-14: error: optimization flag '-falign-jumps=1' is not supported 
> [-Werror,-Wignored-optimization-argument]
>
> When compiled with older Clang, the alignment options are not added, so
> just avoid the complexity for each option.
>
> While here, remove checks for the obsoleted -malign-* forms.
> GCC has supported the preferred -falign-* forms since 3.2.

Thank you for the cleanup. Could you put this in a separate commit 
please (and maybe order it first)?

> Signed-off-by: Fangrui Song <maskray@google.com>
> ---
>   configure.ac | 14 +++-----------
>   1 file changed, 3 insertions(+), 11 deletions(-)

[…]


Kind regards,

Paul