This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [powerpcspe, rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS

From: Tom de Vries <Tom_deVries at mentor dot com>
To: Andrew Jenner <andrew at codesourcery dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Tue, 7 Nov 2017 10:07:46 +0100
Subject: Re: [powerpcspe, rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS
Authentication-results: sourceware.org; auth=none
References: <e374d2e0-89b7-3879-94f7-ec0e8f6d9807@mentor.com>

On 11/07/2017 09:55 AM, Tom de Vries wrote:

Hi,
this patch removes a semicolon after "do {} while (0)" inSUBTARGET_OVERRIDE_OPTIONS (no ChangeLog yet).
Can you build & test this patch (or approve as trivial)?

Hi,

Can you build & test the powerpcspe part of this patch (or approve astrivial)?


Thanks,
- Tom

0003-powerpcspe-rs6000-Remove-semicolon-after-do-while-0-in-SUBTARGET_OVERRIDE_OPTIONS.patch


[powerpcspe, rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS

---
  gcc/config/powerpcspe/aix43.h | 2 +-
  gcc/config/powerpcspe/aix51.h | 2 +-
  gcc/config/powerpcspe/aix52.h | 2 +-
  gcc/config/powerpcspe/aix53.h | 2 +-
  gcc/config/powerpcspe/aix61.h | 2 +-
  gcc/config/powerpcspe/aix71.h | 2 +-
  gcc/config/rs6000/aix43.h     | 2 +-
  gcc/config/rs6000/aix51.h     | 2 +-
  gcc/config/rs6000/aix52.h     | 2 +-
  gcc/config/rs6000/aix53.h     | 2 +-
  gcc/config/rs6000/aix61.h     | 2 +-
  gcc/config/rs6000/aix71.h     | 2 +-
  12 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/gcc/config/powerpcspe/aix43.h b/gcc/config/powerpcspe/aix43.h
index d61956d..bd8a2c0 100644
--- a/gcc/config/powerpcspe/aix43.h
+++ b/gcc/config/powerpcspe/aix43.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/powerpcspe/aix51.h b/gcc/config/powerpcspe/aix51.h
index af7e38e..e9f88df 100644
--- a/gcc/config/powerpcspe/aix51.h
+++ b/gcc/config/powerpcspe/aix51.h
@@ -33,7 +33,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/powerpcspe/aix52.h b/gcc/config/powerpcspe/aix52.h
index 35d2286..eade64c 100644
--- a/gcc/config/powerpcspe/aix52.h
+++ b/gcc/config/powerpcspe/aix52.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/powerpcspe/aix53.h b/gcc/config/powerpcspe/aix53.h
index a1fbd83..b16488b 100644
--- a/gcc/config/powerpcspe/aix53.h
+++ b/gcc/config/powerpcspe/aix53.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/powerpcspe/aix61.h b/gcc/config/powerpcspe/aix61.h
index 0b61505..25fcb62 100644
--- a/gcc/config/powerpcspe/aix61.h
+++ b/gcc/config/powerpcspe/aix61.h
@@ -56,7 +56,7 @@ do {									\
      {									\
        rs6000_current_cmodel = CMODEL_LARGE;				\
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/powerpcspe/aix71.h b/gcc/config/powerpcspe/aix71.h
index 4b986d6..3b8de89 100644
--- a/gcc/config/powerpcspe/aix71.h
+++ b/gcc/config/powerpcspe/aix71.h
@@ -56,7 +56,7 @@ do {									\
      {									\
        rs6000_current_cmodel = CMODEL_LARGE;				\
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix43.h b/gcc/config/rs6000/aix43.h
index d61956d..bd8a2c0 100644
--- a/gcc/config/rs6000/aix43.h
+++ b/gcc/config/rs6000/aix43.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix51.h b/gcc/config/rs6000/aix51.h
index af7e38e..e9f88df 100644
--- a/gcc/config/rs6000/aix51.h
+++ b/gcc/config/rs6000/aix51.h
@@ -33,7 +33,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix52.h b/gcc/config/rs6000/aix52.h
index 35d2286..eade64c 100644
--- a/gcc/config/rs6000/aix52.h
+++ b/gcc/config/rs6000/aix52.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix53.h b/gcc/config/rs6000/aix53.h
index a1fbd83..b16488b 100644
--- a/gcc/config/rs6000/aix53.h
+++ b/gcc/config/rs6000/aix53.h
@@ -39,7 +39,7 @@ do {									\
      {									\
        error ("-maix64 required: 64-bit computation with 32-bit addressing not yet supported"); \
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix61.h b/gcc/config/rs6000/aix61.h
index 0b61505..25fcb62 100644
--- a/gcc/config/rs6000/aix61.h
+++ b/gcc/config/rs6000/aix61.h
@@ -56,7 +56,7 @@ do {									\
      {									\
        rs6000_current_cmodel = CMODEL_LARGE;				\
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"
diff --git a/gcc/config/rs6000/aix71.h b/gcc/config/rs6000/aix71.h
index 4b986d6..3b8de89 100644
--- a/gcc/config/rs6000/aix71.h
+++ b/gcc/config/rs6000/aix71.h
@@ -56,7 +56,7 @@ do {									\
      {									\
        rs6000_current_cmodel = CMODEL_LARGE;				\
      }									\
-} while (0);
+} while (0)

#undef ASM_SPEC

  #define ASM_SPEC "-u %{maix64:-a64 %{!mcpu*:-mppc64}} %(asm_cpu)"

Follow-Ups:
- Re: [powerpcspe, rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS
  - From: Andrew Jenner

References:
- [powerpcspe, rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS
  - From: Tom de Vries

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]