[coreboot-gerrit] New patch to review for coreboot: arch/x86/Makefile.inc: Re-enable -Wa, --divide assembler flag

Andrey Petrov (andrey.petrov@intel.com) gerrit at coreboot.org
Sun Jan 24 03:32:23 CET 2016


Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13299

-gerrit

commit 7509ca3158d7ff2b1f7c8c28fbd63bc2e782562a
Author: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
Date:   Tue Oct 6 14:41:20 2015 -0700

    arch/x86/Makefile.inc: Re-enable -Wa,--divide assembler flag
    
    In the follow-on patches, we use constant division to calculate the
    number of elements in an assembly structure. Since the size of one
    element is not a power of two for all structures, we can't bit shift.
    This allows us to do the computation at assembly time rather than
    runtime.
    
    Change-Id: Ic5693f39dee8daf759bfc16c79ac3c7ddb4de465
    Signed-off-by: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
---
 src/arch/x86/Makefile.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/arch/x86/Makefile.inc b/src/arch/x86/Makefile.inc
index bc4a7d4..825b8e4 100644
--- a/src/arch/x86/Makefile.inc
+++ b/src/arch/x86/Makefile.inc
@@ -102,6 +102,8 @@ endef
 
 ifeq ($(CONFIG_ARCH_BOOTBLOCK_X86_32)$(CONFIG_ARCH_BOOTBLOCK_X86_64),y)
 
+CFLAGS_bootblock+=-Wa,--divide
+
 bootblock-y += boot.c
 bootblock-y += memcpy.c
 bootblock-y += memset.c



More information about the coreboot-gerrit mailing list