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: RFA: cache enabled attribute by insn code

From: Jeff Law <law at redhat dot com>
To: gcc-patches at gcc dot gnu dot org, rdsandiford at googlemail dot com
Date: Fri, 23 May 2014 12:19:18 -0600
Subject: Re: RFA: cache enabled attribute by insn code
Authentication-results: sourceware.org; auth=none
References: <87k39gkgq0 dot fsf at talisman dot default> <537B95E9 dot 1010203 at redhat dot com> <87iop0i13e dot fsf at talisman dot default>

On 05/20/14 15:36, Richard Sandiford wrote:

This is OK for the trunk (referring to the follow-up message which fixed
EWRONGPATCH.


Sorry, while working on the follow-up LRA patch, I realised I hadn't
accounted for target changes that happen directly via target_reinit
(rather than SWITCHABLE_TARGETS) and cases where reinit_regs is called
to change just the register information.  Both could potentially affect
the enabled attribute.

This version adds a recog_init function that clears the data if necessary.
There are no other changes from first time.  Is this still OK?

Thanks for letting me know, that's a minor twiddle -- the patch is still OK.

Jeff

Follow-Ups:
- Re: RFA: cache enabled attribute by insn code
  - From: Christophe Lyon

References:
- RFA: cache enabled attribute by insn code
  - From: Richard Sandiford
- Re: RFA: cache enabled attribute by insn code
  - From: Jeff Law
- Re: RFA: cache enabled attribute by insn code
  - From: Richard Sandiford

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