Re: [PATCH 01/12] s390: remove -fno-strength-reduce flag

Martin Schwidefsky
2019-04-10T15:57:25Z

On Mon, 8 Apr 2019 23:26:14 +0200
Arnd Bergmann <arnd@arndb.de> wrote:

> This was added as a workaround for really old compilers, and it prevents
> building with clang now. I can see no reason for keeping it, as it has
> already been removed for most architectures in the pre-git era, so
> let's remove it everywhere, rather than only for clang.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> arch/s390/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/s390/Makefile b/arch/s390/Makefile
> index 86c76b149cc2..0087d11e3eaf 100644
> --- a/arch/s390/Makefile
> +++ b/arch/s390/Makefile
> @@ -114,7 +114,7 @@ endif
> cfi := $(call as-instr,.cfi_startproc\n.cfi_val_offset 15$(comma)-160\n.cfi_endproc,-DCONFIG_AS_CFI_VAL_OFFSET=1)
>
> KBUILD_CFLAGS += -mbackchain -msoft-float $(cflags-y)
> -KBUILD_CFLAGS += -pipe -fno-strength-reduce -Wno-sign-compare
> +KBUILD_CFLAGS += -pipe -Wno-sign-compare
> KBUILD_CFLAGS += -fno-asynchronous-unwind-tables $(cfi)
> KBUILD_AFLAGS += $(aflags-y) $(cfi)
> export KBUILD_AFLAGS_DECOMPRESSOR

Added to s390/linux:features for the next merge window. Thanks.

--
blue skies,
Martin.

"Reality continues to ruin my life." - Calvin.