This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Speedup by 3.5% by overhauling rtx classes

From: Andreas Jaeger <aj at suse dot de>
To: Bonzini <bonzini at gnu dot org>
Cc: gcc-patches at gcc dot gnu dot org, zack at codesourcery dot com, rth at redhat dot com
Date: Sat, 07 Feb 2004 15:26:23 +0100
Subject: Re: [PATCH] Speedup by 3.5% by overhauling rtx classes
References: <20040207131019.GA3033@fencepost>

Bonzini <bonzini@gnu.org> writes:

> [...]
> @@ -5136,7 +5123,7 @@ simplify_set (rtx x)
>       as long as M1 and M2 have the same number of words.  */
>  
>    if (GET_CODE (src) == SUBREG && subreg_lowpart_p (src)
> -      && GET_RTX_CLASS (GET_CODE (SUBREG_REG (src))) != 'o'
> +      && !OBJECT_P (SUBREG_REG (src))
>        && (((GET_MODE_SIZE (GET_MODE (src)) + (UNITS_PER_WORD - 1))
>  	   / UNITS_PER_WORD)
>  	  == ((GET_MODE_SIZE (GET_MODE (SUBREG_REG (src)))
> @@ -5340,9 +5327,8 @@ simplify_logical (rtx x, int last)
>  	  if (GET_CODE (x) != AND)
>  	    return x;
>  
> -	  if (GET_RTX_CLASS (GET_CODE (x)) == 'c'
> -	      || GET_RTX_CLASS (GET_CODE (x)) == '2')
> -	    op0 = XEXP (x, 0), op1 = XEXP (x, 1);
> +	  op0 = XEXP (x, 0);
> +	  op1 = XEXP (x, 1);
>  	}

This hunk looks wrong.  Why do you remove the complete if?

Andreas
-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
  SuSE Linux AG, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

Attachment: pgp00000.pgp
Description: PGP signature

Follow-Ups:
- Re: [PATCH] Speedup by 3.5% by overhauling rtx classes
  - From: Andreas Schwab

References:
- [PATCH] Speedup by 3.5% by overhauling rtx classes
  - From: Bonzini

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]