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, reload] PRE_INC with invalid hard reg

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Alan Modra <amodra at gmail dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Fri, 12 Feb 2016 12:52:32 +0100
Subject: Re: [PATCH, reload] PRE_INC with invalid hard reg
Authentication-results: sourceware.org; auth=none
References: <20160211094540 dot GB10888 at bubble dot grove dot modra dot org> <56BC9AB1 dot 9010205 at redhat dot com> <20160212032757 dot GE10888 at bubble dot grove dot modra dot org>

On 02/12/2016 04:27 AM, Alan Modra wrote:

I don't understand this comment.  If we're pushing a reload of the
inner reg, then the SECONDARY_MEMORY_NEEDED code in push_reload will
fire.  Why then should there be any need to do anything special in
find_reloads_address_1 regarding secondary memory?

The idea was to find out whether we needed to special case things. But,after looking at this area for a while, I think your original patchmakes the most sense and the things I was worried about are non-issues.Please commit.



Bernd

References:
- [PATCH, reload] PRE_INC with invalid hard reg
  - From: Alan Modra
- Re: [PATCH, reload] PRE_INC with invalid hard reg
  - From: Bernd Schmidt
- Re: [PATCH, reload] PRE_INC with invalid hard reg
  - From: Alan Modra

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