arm.c (arm_print_operand case 'S'): Validate that the operand is a shift operand…

arm.c (arm_print_operand case 'S'): Validate that the operand is a shift operand before calling shift_op.

	* arm.c (arm_print_operand case 'S'): Validate that the operand is
	a shift operand before calling shift_op.  Avoid redundant call of
	shift_op.

From-SVN: r114264

gcc/ChangeLog

+2006-05-31  Richard Earnshaw  <richard.earnshaw@arm.com>
+
+	* arm.c (arm_print_operand case 'S'): Validate that the operand is
+	a shift operand before calling shift_op.  Avoid redundant call of
+	shift_op.
+
 2006-05-30  Naveen.H.S  <naveenh@kpitcummins.com>
 	    DJ Delorie  <dj@redhat.com>
 

gcc/config/arm/arm.c

@@ -11013,11 +11013,19 @@ arm_print_operand (FILE *stream, rtx x, int code)
     case 'S':
       {
 	HOST_WIDE_INT val;
-	const char * shift = shift_op (x, &val);
+	const char *shift;
+
+	if (!shift_operator (x, SImode))
+	  {
+	    output_operand_lossage ("invalid shift operand");
+	    break;
+	  }
+
+	shift = shift_op (x, &val);
 
 	if (shift)
 	  {
-	    fprintf (stream, ", %s ", shift_op (x, &val));
+	    fprintf (stream, ", %s ", shift);
 	    if (val == -1)
 	      arm_print_operand (stream, XEXP (x, 1), 0);
 	    else