Patchwork [ARM] : Remove superfluous test in arm_function_arg

mail settings
Submitter John Tytgat
Date July 15, 2010, 8:35 p.m.
Message ID <>
Download mbox | patch
Permalink /patch/59043/
State New
Headers show


John Tytgat - July 15, 2010, 8:35 p.m.
Attached patch removes in arm_function_arg() a superfluous test on one
of its parameters as that test is already done at the beginning of the
routine and the parameter doesn't change during the function.

John Tytgat  <>

	* config/arm/arm.c (arm_function_arg): Remove superfluous test.

Please commit after positive review.



Index: config/arm/arm.c
--- config/arm/arm.c	(revision 162199)
+++ config/arm/arm.c	(working copy)
@@ -4479,10 +4479,6 @@ 
       && arm_needs_doubleword_align (mode, type))
-  if (mode == VOIDmode)
-    /* Pick an arbitrary value for operand 2 of the call insn.  */
-    return const0_rtx;
   /* Only allow splitting an arg between regs and memory if all preceding
      args were allocated to regs.  For args passed by reference we only count
      the reference pointer.  */