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: Extend list of codes accepted by address_operand

From: Jeff Law <law at redhat dot com>
To: gcc-patches at gcc dot gnu dot org, richard dot sandiford at arm dot com
Date: Mon, 27 Apr 2015 12:14:16 -0600
Subject: Re: Extend list of codes accepted by address_operand
Authentication-results: sourceware.org; auth=none
References: <87mw1tyjdx dot fsf at e105548-lin dot cambridge dot arm dot com>

On 04/27/2015 04:07 AM, Richard Sandiford wrote:

While testing the patch that I'm about to post, I found that the list
of address_operand codes in gensupport.c was incomplete.  This patch
fills in some "obvious" missing entries.

It isn't exactly clear which codes are allowed at the top level of
an address.  decompose_address is known to be general enough for all LRA
targets, but it also technically accepts any unary operator as the outer
code, whereas I think only SIGN_EXTEND and ZERO_EXTEND are used at the
top level.  The routine accepts ZERO_EXTRACT and SIGN_EXTRACT, but I
think in practice they're only used for index values and not full addresses.

Tested on x86_64-linux-gnu.  OK to install?

Thanks,
Richard


gcc/
	* gensupport.c (std_preds): Add missing codes to address_operand entry.

OK.
jeff

References:
- Extend list of codes accepted by address_operand
  - From: Richard Sandiford

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