[coreboot] New patch to review for coreboot: 74287ab Teach romcc about attribute((unused))

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Fri Jun 22 02:23:11 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/1132

-gerrit

commit 74287ab10266bef71fd21ced6e5857ef6ce16182
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Thu Jun 21 17:21:08 2012 -0700

    Teach romcc about attribute((unused))
    
    This makes it easier to use the same code on romcc and gcc.
    Specifying attribute((unused)) on romcc does nothing.
    
    Change-Id: If9a6900cad12900e499c4b8c91586511eb801987
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/romcc/romcc.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c
index 1a939fb..b0f6198 100644
--- a/util/romcc/romcc.c
+++ b/util/romcc/romcc.c
@@ -1084,6 +1084,7 @@ struct compile_state {
 	struct hash_entry *i_default;
 	struct hash_entry *i_return;
 	struct hash_entry *i_noreturn;
+	struct hash_entry *i_unused;
 	/* Additional hash entries for predefined macros */
 	struct hash_entry *i_defined;
 	struct hash_entry *i___VA_ARGS__;
@@ -12752,6 +12753,9 @@ static unsigned int attrib(struct compile_state *state, unsigned int attributes)
 		else if (ident == state->i_noreturn) {
 			// attribute((noreturn)) does nothing (yet?)
 		}
+		else if (ident == state->i_unused) {
+			// attribute((unused)) does nothing (yet?)
+		}
 		else {
 			error(state, 0, "Unknown attribute:%s", ident->name);
 		}
@@ -24988,6 +24992,7 @@ static void compile(const char *filename,
 	state.i_noinline      = lookup(&state, "noinline", 8);
 	state.i_always_inline = lookup(&state, "always_inline", 13);
 	state.i_noreturn      = lookup(&state, "noreturn", 8);
+	state.i_unused        = lookup(&state, "unused", 8);
 
 	/* Process the command line macros */
 	process_cmdline_macros(&state);




More information about the coreboot mailing list