Patchwork [ARM] : Remove superfluous test in arm_function_arg

login
register
mail settings
Submitter John Tytgat
Date July 15, 2010, 8:35 p.m.
Message ID <678f613751.Jo@hobbes.bass-software.com>
Download mbox | patch
Permalink /patch/59043/
State New
Headers show

Comments

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  <John.Tytgat@aaug.net>

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

Please commit after positive review.

John.

Patch

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))
     pcum->nregs++;
 
-  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.  */