This member seemed to be used for: - current value of an enum during parsing - index into the virtual frame but is currently not used and never set, it's only displayed in compile-i386 & show_parse (as zero since never initialized). Remove it to make struct symbol a bit more light. Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> --- compile-i386.c | 2 +- show-parse.c | 2 +- symbol.h | 1 - 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/compile-i386.c b/compile-i386.c index 44b72ec39..633d9a476 100644 --- a/compile-i386.c +++ b/compile-i386.c @@ -2238,7 +2238,7 @@ static struct storage *x86_symbol_expr(struct symbol *sym) return new; } if (sym->ctype.modifiers & MOD_ADDRESSABLE) { - printf("\taddi.%d\t\tv%d,vFP,$%lld\n", bits_in_pointer, new->pseudo, sym->value); + printf("\taddi.%d\t\tv%d,vFP,$%lld\n", bits_in_pointer, new->pseudo, 0ULL); return new; } printf("\taddi.%d\t\tv%d,vFP,$offsetof(%s:%p)\n", bits_in_pointer, new->pseudo, show_ident(sym->ident), sym); diff --git a/show-parse.c b/show-parse.c index d365d737f..dd7e76fc4 100644 --- a/show-parse.c +++ b/show-parse.c @@ -927,7 +927,7 @@ static int show_symbol_expr(struct symbol *sym) return new; } if (sym->ctype.modifiers & MOD_ADDRESSABLE) { - printf("\taddi.%d\t\tv%d,vFP,$%lld\n", bits_in_pointer, new, sym->value); + printf("\taddi.%d\t\tv%d,vFP,$%lld\n", bits_in_pointer, new, 0ULL); return new; } printf("\taddi.%d\t\tv%d,vFP,$offsetof(%s:%p)\n", bits_in_pointer, new, show_ident(sym->ident), sym); diff --git a/symbol.h b/symbol.h index ac3fd8bf0..1bedc75c3 100644 --- a/symbol.h +++ b/symbol.h @@ -182,7 +182,6 @@ struct symbol { struct symbol_list *inline_symbol_list; struct expression *initializer; struct entrypoint *ep; - long long value; /* Initial value */ struct symbol *definition; }; }; -- 2.13.0 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html