[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] configure: Check -falign-jumps=1 -falign-functions=1 beside
From: |
Fāng-ruì Sòng |
Subject: |
Re: [PATCH] configure: Check -falign-jumps=1 -falign-functions=1 beside -falign-loops=1 |
Date: |
Mon, 23 Aug 2021 10:54:28 -0700 |
Ping.
This is needed to allow Clang 14.0.0 to build grub.
On Mon, Aug 16, 2021 at 12:06 PM Fangrui Song <maskray@google.com> wrote:
>
> Clang 14.0.0 has implemented -falign-loops=1 but -falign-jumps=1 is
> still unsupported. The check should test all flags which may be added
> into CFLAGS.
>
> Signed-off-by: Fangrui Song <maskray@google.com>
> ---
> configure.ac | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index bee28dbeb..b2fd0c3e4 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -799,14 +799,14 @@ fi
> # Force no alignment to save space on i386.
> if test "x$target_cpu" = xi386; then
> AC_CACHE_CHECK([whether -falign-loops works], [grub_cv_cc_falign_loop], [
> - CFLAGS="$TARGET_CFLAGS -falign-loops=1 -Werror"
> + CFLAGS="$TARGET_CFLAGS -falign-jumps=1 -falign-loops=1
> -falign-functions=1 -Werror"
> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[]])],
> [grub_cv_cc_falign_loop=yes],
> [grub_cv_cc_falign_loop=no])
> ])
>
> AC_CACHE_CHECK([whether -malign-loops works], [grub_cv_cc_malign_loop], [
> - CFLAGS="$TARGET_CFLAGS -malign-loops=1 -Werror"
> + CFLAGS="$TARGET_CFLAGS -malign-jumps=1 -malign-loops=1
> -malign-functions=1 -Werror"
> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[]])],
> [grub_cv_cc_malign_loop=yes],
> [grub_cv_cc_malign_loop=no])
> --
> 2.33.0.rc1.237.g0d66db33f3-goog
>
--
宋方睿