[coreboot] New patch to review for coreboot: 0afabb9 Enable -Werror for romcc

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Sat Mar 31 00:15:18 CEST 2012


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/818

-gerrit

commit 0afabb90a9f893e37097e0c393efe2d3f1f4cd94
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Fri Mar 30 12:11:04 2012 -0700

    Enable -Werror for romcc
    
    ... and remove some dead code.
    
    Change-Id: Id959bdf57af09db2a1f5742555c2dcabca38ac9a
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/romcc/Makefile |    2 +-
 util/romcc/romcc.c  |    4 ----
 2 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/util/romcc/Makefile b/util/romcc/Makefile
index 6543fbb..8242eb5 100644
--- a/util/romcc/Makefile
+++ b/util/romcc/Makefile
@@ -1,7 +1,7 @@
 # Move the configuration defines to makefile.conf
 CC=gcc
 CPPFLAGS=
-CFLAGS= -g -Wall $(CPPFLAGS)
+CFLAGS= -g -Wall -Werror $(CPPFLAGS)
 CPROF_FLAGS=-pg -fprofile-arcs
 
 all: romcc test
diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c
index c7ef223..1a939fb 100644
--- a/util/romcc/romcc.c
+++ b/util/romcc/romcc.c
@@ -9160,10 +9160,6 @@ static struct triple *decompose_index(struct compile_state *state,
 static void decompose_compound_types(struct compile_state *state)
 {
 	struct triple *ins, *next, *first;
-#if DEBUG_DECOMPOSE_HIRES
-	FILE *fp;
-	fp = state->dbgout;
-#endif
 	first = state->first;
 	ins = first;
 




More information about the coreboot mailing list