Commit 1416d1c4 authored Apr 10, 2017 by Greg Hackmann Committed by Amit Pundir Dec 18, 2017

ANDROID: HACK: arm64: use -mno-implicit-float instead of -mgeneral-regs-only



LLVM bug 30792 causes clang's AArch64 backend to crash compiling
arch/arm64/crypto/aes-ce-cipher.c.  Replacing -mgeneral-regs-only with
-mno-implicit-float is the suggested workaround.

Drop this patch once the clang bug has been fixed.

Change-Id: I89fda12d826783ffe18142ba048ff10bd769e240
Signed-off-by: Greg Hackmann <ghackmann@google.com>
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>

[AmitP: Refactored the changes to align with changes from upstream commit
        bbb56c27 ("arm64: Add detection code for broken .inst support in binutils")]
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>

parent b9b96462

arch/arm64/Makefile

+8 −1

Original line number	Diff line number	Diff line
		@@ -45,7 +45,14 @@ $(warning Detected assembler with broken .inst; disassembly will be unreliable)
		endif
		endif

		KBUILD_CFLAGS += -mgeneral-regs-only $(lseinstr) $(brokengasinst)
		ifeq ($(cc-name),clang)
		# This is a workaround for https://bugs.llvm.org/show_bug.cgi?id=30792.
		# TODO: revert when this is fixed in LLVM.
		KBUILD_CFLAGS += -mno-implicit-float
		else
		KBUILD_CFLAGS += -mgeneral-regs-only
		endif
		KBUILD_CFLAGS += $(lseinstr) $(brokengasinst)
		KBUILD_CFLAGS += -fno-asynchronous-unwind-tables
		KBUILD_CFLAGS += $(call cc-option, -mpc-relative-literal-loads)
		KBUILD_CFLAGS += -fno-pic