New patch to review for coreboot: a68969f vendorcode/amd/agesa/f14/Makefile.inc: Remove `-mtune` from `AGESA_FLAGS`

[Paul Menzel]

Paul Menzel ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5563

-gerrit

commit a68969f1b11f323c7ac813616236b720550065f2
Author: Paul Menzel 
Date:   Mon Apr 21 23:14:52 2014 +0200

vendorcode/amd/agesa/f14/Makefile.inc: Remove `-mtune` from `AGESA_FLAGS`

Specifying the same CPU type in the switch `mtune` as in the switch
    `march` does not make any sense.

-march=k8-sse3 -mtune=k8-sse3

From chapter 3.17.17 of the GCC manual [1]:

-mtune=cpu-type
    	Tune to cpu-type everything applicable about the generated code,
    	except for the ABI and the set of available instructions. While
    	picking a specific cpu-type schedules things appropriately for
    	that particular chip, the compiler does not generate any code
    	that cannot run on the default machine type unless you use a
    	-march=cpu-type option.

[1] http://gcc.gnu.org/onlinedocs/gcc/i386-and-x86-64-Options.html#i386-and-x86-...
        3.17.17 Intel 386 and AMD x86-64 Options

Change-Id: If51fca03f4137a2937fb6c1ad5dd698c6a3a4d8f
    Signed-off-by: Paul Menzel 
---
 src/vendorcode/amd/agesa/f14/Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vendorcode/amd/agesa/f14/Makefile.inc b/src/vendorcode/amd/agesa/f14/Makefile.inc
index f457277..2e7853c 100644
--- a/src/vendorcode/amd/agesa/f14/Makefile.inc
+++ b/src/vendorcode/amd/agesa/f14/Makefile.inc
@@ -62,7 +62,7 @@ AGESA_INC += -I$(AGESA_ROOT)/Proc/Recovery/GNB
 AGESA_INC += -I$(AGESA_ROOT)/Proc/Recovery/CPU
 AGESA_INC += -I$(AGESA_ROOT)/Proc/Recovery/Mem

-AGESA_CFLAGS =-march=k8-sse3 -mtune=k8-sse3 -fno-zero-initialized-in-bss -fno-strict-aliasing
+AGESA_CFLAGS =-march=k8-sse3 -fno-zero-initialized-in-bss -fno-strict-aliasing

export AGESA_ROOT := $(AGESA_ROOT)
 export AGESA_INC  := $(AGESA_INC)