From 546140e2ea97fba525baeee93054aefd0b6edc4b Mon Sep 17 00:00:00 2001 From: bje Date: Mon, 1 May 2006 09:21:46 +0000 Subject: [PATCH] * listing.c (listing_listing): Remove useless loop. * macro.c (macro_expand): Remove is_positional local variable. * read.c (s_comm_internal): Simplify `if' condition 1 || x -> 1 and simplify surrounding expressions, where possible. (assign_symbol): Likewise. (s_weakref): Likewise. * symbols.c (colon): Likewise. --- gas/ChangeLog | 10 ++++++++++ gas/listing.c | 9 --------- gas/macro.c | 3 --- gas/read.c | 19 +++++-------------- gas/symbols.c | 5 +---- 5 files changed, 16 insertions(+), 30 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 263654ab8c..3ef40aba49 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,13 @@ +2006-05-01 Ben Elliston + + * listing.c (listing_listing): Remove useless loop. + * macro.c (macro_expand): Remove is_positional local variable. + * read.c (s_comm_internal): Simplify `if' condition 1 || x -> 1 + and simplify surrounding expressions, where possible. + (assign_symbol): Likewise. + (s_weakref): Likewise. + * symbols.c (colon): Likewise. + 2006-05-01 James Lemke * subsegs.c (subseg_set_rest): Always set seginfp->frchainP if NULL. diff --git a/gas/listing.c b/gas/listing.c index 61ef6f55e4..717ed60111 100644 --- a/gas/listing.c +++ b/gas/listing.c @@ -946,15 +946,6 @@ listing_listing (char *name ATTRIBUTE_UNUSED) buffer = xmalloc (listing_rhs_width); data_buffer = xmalloc (MAX_BYTES); eject = 1; - list = head; - - while (list != (list_info_type *) NULL && 0) - { - if (list->next) - list->frag = list->next->frag; - list = list->next; - } - list = head->next; while (list) diff --git a/gas/macro.c b/gas/macro.c index af98bada6a..e6e9d05ba2 100644 --- a/gas/macro.c +++ b/gas/macro.c @@ -1025,7 +1025,6 @@ macro_expand (int idx, sb *in, macro_entry *m, sb *out) sb t; formal_entry *ptr; formal_entry *f; - int is_positional = 0; int is_keyword = 0; int narg = 0; const char *err = NULL; @@ -1116,8 +1115,6 @@ macro_expand (int idx, sb *in, macro_entry *m, sb *out) } else { - /* This is a positional arg. */ - is_positional = 1; if (is_keyword) { err = _("can't mix positional and keyword arguments"); diff --git a/gas/read.c b/gas/read.c index 0485d72dec..d51835e3ce 100644 --- a/gas/read.c +++ b/gas/read.c @@ -1,6 +1,6 @@ /* read.c - read a source file - Copyright 1986, 1987, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, - 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005 + 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006 Free Software Foundation, Inc. This file is part of GAS, the GNU Assembler. @@ -1478,10 +1478,7 @@ s_comm_internal (int param, ignore_rest_of_line (); goto out; } - /* This could be avoided when the symbol wasn't used so far, but - the comment in struc-symbol.h says this flag isn't reliable. */ - if (1 || !symbol_used_p (symbolP)) - symbolP = symbol_clone (symbolP, 1); + symbolP = symbol_clone (symbolP, 1); S_SET_SEGMENT (symbolP, undefined_section); S_SET_VALUE (symbolP, 0); symbol_set_frag (symbolP, &zero_address_frag); @@ -2812,10 +2809,7 @@ assign_symbol (char *name, int mode) /* If the symbol is volatile, copy the symbol and replace the original with the copy, so that previous uses of the symbol will retain the value of the symbol at the point of use. */ - else if (S_IS_VOLATILE (symbolP) - /* This could be avoided when the symbol wasn't used so far, but - the comment in struc-symbol.h says this flag isn't reliable. */ - && (1 || symbol_used_p (symbolP))) + else if (S_IS_VOLATILE (symbolP)) symbolP = symbol_clone (symbolP, 1); } @@ -3186,17 +3180,14 @@ s_weakref (int ignore ATTRIBUTE_UNUSED) if (S_IS_DEFINED (symbolP) || symbol_equated_p (symbolP)) { - if(!S_IS_VOLATILE (symbolP)) + if (!S_IS_VOLATILE (symbolP)) { as_bad (_("symbol `%s' is already defined"), name); *end_name = delim; ignore_rest_of_line (); return; } - /* This could be avoided when the symbol wasn't used so far, but - the comment in struc-symbol.h says this flag isn't reliable. */ - if (1 || !symbol_used_p (symbolP)) - symbolP = symbol_clone (symbolP, 1); + symbolP = symbol_clone (symbolP, 1); S_CLEAR_VOLATILE (symbolP); } diff --git a/gas/symbols.c b/gas/symbols.c index ca2d41ea9c..499e8c4d31 100644 --- a/gas/symbols.c +++ b/gas/symbols.c @@ -336,10 +336,7 @@ colon (/* Just seen "x:" - rattle symbols & frags. */ || S_IS_COMMON (symbolP) || S_IS_VOLATILE (symbolP)) { - if (S_IS_VOLATILE (symbolP) - /* This could be avoided when the symbol wasn't used so far, but - the comment in struc-symbol.h says this flag isn't reliable. */ - && (1 || !symbol_used_p (symbolP))) + if (S_IS_VOLATILE (symbolP)) { symbolP = symbol_clone (symbolP, 1); S_SET_VALUE (symbolP, 0); -- 2.11.0