diff mbox series

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

Message ID e374d2e0-89b7-3879-94f7-ec0e8f6d9807@mentor.com
State New
Headers show
Series [powerpcspe,rs6000] Remove semicolon after do {} while (0) in SUBTARGET_OVERRIDE_OPTIONS | expand

Commit Message

Tom de Vries Nov. 7, 2017, 8:55 a.m. UTC 
Hi,

this patch removes a semicolon after "do {} while (0)" in 
SUBTARGET_OVERRIDE_OPTIONS (no ChangeLog yet).

Can you build & test this patch (or approve as trivial)?

Thanks,
- Tom

Comments

Segher Boessenkool Nov. 7, 2017, 9:05 a.m. UTC | #1 
On Tue, Nov 07, 2017 at 09:55:39AM +0100, Tom de Vries wrote:
> this patch removes a semicolon after "do {} while (0)" in 
> SUBTARGET_OVERRIDE_OPTIONS (no ChangeLog yet).
> 
> Can you build & test this patch (or approve as trivial)?

The rs6000 parts are okay for trunk (I'll take all blame if it breaks, heh).
Thanks!


Segher
Tom de Vries Nov. 7, 2017, 9:07 a.m. UTC | #2 
On 11/07/2017 09:55 AM, Tom de Vries wrote:
> Hi,
> 
> this patch removes a semicolon after "do {} while (0)" in 
> SUBTARGET_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 as 
trivial)?

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)"
>
Andrew Jenner Nov. 7, 2017, 10:38 a.m. UTC | #3 
Hi Tom,

On 07/11/2017 09:07, Tom de Vries wrote:
> On 11/07/2017 09:55 AM, Tom de Vries wrote:
>> Hi,
>>
>> this patch removes a semicolon after "do {} while (0)" in 
>> SUBTARGET_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 as 
> trivial)?

Approved. Thanks!

Andrew
diff mbox series

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)"