[Patch ARM] Fix typo in *<arith_shift_insn>_shiftsi

Ramana Radhakrishnan ramana.radhakrishnan@arm.com
Wed Nov 12 17:11:00 GMT 2014


As per RichardS's comment on the checkin to change the patterns for 
<arith_shift_insn>_shiftsi here. 
https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02534.html

I toyed with the idea of removing this check but on balance it's 
probably safe to have this to have the condition fail before the 
recognizer kicks in.

Bootstrapped and regression tested on armhf.

Applied to trunk.

Ramana

2014-11-12  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

         * config/arm/arm.md (arith_shift_insn): Fix typo in operand
         number.
-------------- next part --------------
Index: gcc/config/arm/arm.h
===================================================================
--- gcc/config/arm/arm.h	(revision 216854)
+++ gcc/config/arm/arm.h	(working copy)
@@ -36,7 +36,7 @@
 #define MACHMODE int
 #else
 #include "insn-modes.h"
-#define MACHMODE enum machine_mode
+#define MACHMODE machine_mode
 #endif
 
 #include "config/vxworks-dummy.h"
@@ -1510,7 +1510,7 @@
 }
 arm_stack_offsets;
 
-#if !defined(GENERATOR_FILE) && !defined(USED_FOR_TARGET)
+#if !defined(GENERATOR_FILE) && !defined (USED_FOR_TARGET)
 /* A C structure for machine-specific, per-function data.
    This is added to the cfun structure.  */
 typedef struct GTY(()) machine_function
@@ -1578,6 +1578,7 @@
 /* Default procedure calling standard of current compilation unit. */
 extern enum arm_pcs arm_pcs_default;
 
+#if !defined (USED_FOR_TARGET)
 /* A C type for declaring a variable that is used as the first argument of
    `FUNCTION_ARG' and other related values.  */
 typedef struct
@@ -1611,6 +1612,7 @@
   int aapcs_vfp_rcount;
   MACHMODE aapcs_vfp_rmode;
 } CUMULATIVE_ARGS;
+#endif
 
 #define FUNCTION_ARG_PADDING(MODE, TYPE) \
   (arm_pad_arg_upward (MODE, TYPE) ? upward : downward)


More information about the Gcc-patches mailing list