public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] make a few variables static
@ 2016-04-13  8:13 tbsaunde+binutils
  2016-04-13 10:04 ` Nick Clifton
  0 siblings, 1 reply; 2+ messages in thread
From: tbsaunde+binutils @ 2016-04-13  8:13 UTC (permalink / raw)
  To: binutils; +Cc: Trevor Saunders

From: Trevor Saunders <tbsaunde+binutils@tbsaunde.org>

Hi,

They are only used in one file, so we might as well restrict there scope to
that file, and theoretically this might slightly improve compilers ability to
optimize usage of these variables.

built crosses to sparc-elf, ppc64-elf, and nios2-elf, ok?
Trev

gas/ChangeLog:

2016-04-13  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

	* config/tc-nios2.c (nios2_as_options): Make file static.
	* config/tc-ppc.c (toc_reloc_ypes): Likewise.
	* config/tc-sparc.c (native_op_table): Likewise.
---
 gas/config/tc-nios2.c | 2 +-
 gas/config/tc-ppc.c   | 2 +-
 gas/config/tc-sparc.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gas/config/tc-nios2.c b/gas/config/tc-nios2.c
index bf37ff7..d207758 100644
--- a/gas/config/tc-nios2.c
+++ b/gas/config/tc-nios2.c
@@ -98,7 +98,7 @@ typedef enum
 } relax_optionT;
 
 /* Struct contains all assembler options set with .set.  */
-struct
+static struct
 {
   /* .set noat -> noat = 1 allows assembly code to use at without warning
      and macro expansions generate a warning.
diff --git a/gas/config/tc-ppc.c b/gas/config/tc-ppc.c
index 1d2c57f..8bfdfdc 100644
--- a/gas/config/tc-ppc.c
+++ b/gas/config/tc-ppc.c
@@ -207,7 +207,7 @@ ppc_cpu_t sticky = 0;
 unsigned int ppc_abiversion = 0;
 
 /* Flags set on encountering toc relocs.  */
-enum {
+static enum {
   has_large_toc_reloc = 1,
   has_small_toc_reloc = 2
 } toc_reloc_types;
diff --git a/gas/config/tc-sparc.c b/gas/config/tc-sparc.c
index a51ca1f..b956131 100644
--- a/gas/config/tc-sparc.c
+++ b/gas/config/tc-sparc.c
@@ -758,7 +758,7 @@ md_show_usage (FILE *stream)
 }
 \f
 /* Native operand size opcode translation.  */
-struct
+static struct
   {
     const char *name;
     const char *name32;
-- 
2.1.4

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] make a few variables static
  2016-04-13  8:13 [PATCH] make a few variables static tbsaunde+binutils
@ 2016-04-13 10:04 ` Nick Clifton
  0 siblings, 0 replies; 2+ messages in thread
From: Nick Clifton @ 2016-04-13 10:04 UTC (permalink / raw)
  To: tbsaunde+binutils, binutils

Hi Trev,

> 2016-04-13  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
> 
> 	* config/tc-nios2.c (nios2_as_options): Make file static.
> 	* config/tc-ppc.c (toc_reloc_ypes): Likewise.
> 	* config/tc-sparc.c (native_op_table): Likewise.
 
Approved - please apply.

Cheers
  Nick

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-04-13 10:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-13  8:13 [PATCH] make a few variables static tbsaunde+binutils
2016-04-13 10:04 ` Nick Clifton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).