[PATCH] Use -mgeneral-regs-only whenever possible

25 views
Skip to first unread message

Jan Kiszka

unread,
Aug 26, 2023, 4:59:32 AM8/26/23
to efibootguard-dev, Peter Marko
From: Jan Kiszka <jan.k...@siemens.com>

This obsoletes explicit no-sse/mmx and resolves build issue with custom
toolchains on x86, plus it simplifies the setup.

Along that, clarify why we need CFLAGS_MGENERAL_REGS_ONLY.

Signed-off-by: Jan Kiszka <jan.k...@siemens.com>
---
Makefile.am | 7 +------
configure.ac | 1 +
2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index aefdada..02e9cac 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -216,18 +216,13 @@ efi_cflags = \
-Wsign-compare \
-DGNU_EFI_USE_MS_ABI \
-Werror \
+ $(CFLAGS_MGENERAL_REGS_ONLY) \
$(CFLAGS)

if ARCH_X86_64
efi_cflags += \
- -mno-sse \
- -mno-mmx \
-mno-red-zone
endif
-if ARCH_ARM
-efi_cflags += \
- $(CFLAGS_MGENERAL_REGS_ONLY)
-endif

efi_ldflags = \
-T $(GNUEFI_LIB_DIR)/elf_$(ARCH)_efi.lds \
diff --git a/configure.ac b/configure.ac
index d448d71..b000603 100644
--- a/configure.ac
+++ b/configure.ac
@@ -111,6 +111,7 @@ AC_SUBST([ARCH])
AC_SUBST([MACHINE_TYPE_NAME])
AM_CONDITIONAL([ARCH_IS_X86], [test "$ARCH" = "ia32" -o "$ARCH" = "x86_64"])

+# -mgeneral-regs-only was introduced with gcc-9 to ARM, and RISCV64 does not support it until now
AX_CHECK_COMPILE_FLAG([-mgeneral-regs-only],
[CFLAGS_MGENERAL_REGS_ONLY=-mgeneral-regs-only],
[CFLAGS_MGENERAL_REGS_ONLY=])
--
2.35.3

JEMS EBERHARD HORBEL

unread,
Dec 9, 2023, 2:05:09 PM12/9/23
to EFI Boot Guard
DIRECT SENDER IS HERE LETS DEAL.

JENS EBERHARD



MT103/202 DIRECT WIRE TRANSFER
PAYPAL TRANSFER
CASHAPP TRANSFER
ZELLE TRANSFER
TRANSFER WISE
WESTERN UNION TRANSFER
BITCOIN FLASHING 
BANK ACCOUNT LOADING/FLASHING
IBAN TO IBAN TRANSFER
MONEYGRAM TRANSFER
SLBC PROVIDER
CREDIT CARD TOP UP
SEPA TRANSFER
WIRE TRANSFER
GLOBALPAY INC US

Thanks.


NOTE; ONLY SERIOUS / RELIABLE RECEIVERS CAN CONTACT.

DM ME ON WHATSAPP FOR A SERIOUS DEAL.

+447405129573

Reply all
Reply to author
Forward
0 new messages