diff mbox series

[arm,committed] Remove semicolon after while {} do (0) in HANDLE_NARROW_SHIFT_ARITH

Message ID 2a8c2f15-29e1-e58b-970e-2d0a6bfa7a06@mentor.com
State New
Headers show
Series [arm,committed] Remove semicolon after while {} do (0) in HANDLE_NARROW_SHIFT_ARITH | expand

Commit Message

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

this patch removes a semicolon after "while {} do (0)" in 
HANDLE_NARROW_SHIFT_ARITH. This allows the macro to be used in 
if-then-elses without curly braces.

Build arm compiler.

Committed as obvious.

Thanks,
- Tom

Comments

Kyrill Tkachov Nov. 7, 2017, 9:48 a.m. UTC | #1 
On 07/11/17 08:19, Tom de Vries wrote:
> Hi,
>
> this patch removes a semicolon after "while {} do (0)" in
> HANDLE_NARROW_SHIFT_ARITH. This allows the macro to be used in
> if-then-elses without curly braces.
>
> Build arm compiler.
>
> Committed as obvious.
>

Thanks for fixing this Tom.

Kyrill

> Thanks,
> - Tom
diff mbox series

Patch

[arm] Remove semicolon after while {} do (0) in HANDLE_NARROW_SHIFT_ARITH

2017-11-06  Tom de Vries  <tom@codesourcery.com>

	PR other/82784
	* config/arm/arm.c (HANDLE_NARROW_SHIFT_ARITH): Remove semicolon after
	"while {} do (0)".
	(arm_rtx_costs_internal): Add missing semicolon after
	HANDLE_NARROW_SHIFT_ARITH call.

---
 gcc/config/arm/arm.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 033ec25..c2a3e9c 100644
--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -9222,7 +9222,7 @@  arm_unspec_cost (rtx x, enum rtx_code /* outer_code */, bool speed_p, int *cost)
 	        return true;						\
 	      }								\
 	  }								\
-	while (0);
+	while (0)
 
 /* RTX costs.  Make an estimate of the cost of executing the operation
    X, which is contained with an operation with code OUTER_CODE.
@@ -9654,8 +9654,8 @@  arm_rtx_costs_internal (rtx x, enum rtx_code code, enum rtx_code outer_code,
 	  /* We check both sides of the MINUS for shifter operands since,
 	     unlike PLUS, it's not commutative.  */
 
-	  HANDLE_NARROW_SHIFT_ARITH (MINUS, 0)
-	  HANDLE_NARROW_SHIFT_ARITH (MINUS, 1)
+	  HANDLE_NARROW_SHIFT_ARITH (MINUS, 0);
+	  HANDLE_NARROW_SHIFT_ARITH (MINUS, 1);
 
 	  /* Slightly disparage, as we might need to widen the result.  */
 	  *cost += 1;
@@ -9766,7 +9766,7 @@  arm_rtx_costs_internal (rtx x, enum rtx_code code, enum rtx_code outer_code,
 	  rtx shift_op, shift_reg;
 	  shift_reg = NULL;
 
-	  HANDLE_NARROW_SHIFT_ARITH (PLUS, 0)
+	  HANDLE_NARROW_SHIFT_ARITH (PLUS, 0);
 
 	  if (CONST_INT_P (XEXP (x, 1)))
 	    {