Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-25 Thread Steven Bosscher
On Wed, Sep 25, 2013 at 4:55 PM, Vladimir Makarov wrote: > On 09/24/2013 03:40 PM, Mike Stump wrote: >> On Sep 24, 2013, at 12:23 PM, Steven Bosscher wrote: >>> On Tue, Sep 24, 2013 at 5:03 PM, Eric Botcazou wrote: > This patch removes REG_DEAD and REG_UNUSED notes DF framework is supposed

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-25 Thread Eric Botcazou
> > The description is too terse. In the RTL middle-end, you shouldn't have > > to manually deal with the REG_DEAD and REG_UNUSED notes (unlike > > REG_EQUAL and REG_EQUIV notes), as the DF framework is supposed to do > > it for you. > > Unfortunately LRA uses its own DF framework. In fact reload

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-25 Thread Vladimir Makarov
On 09/24/2013 03:40 PM, Mike Stump wrote: > On Sep 24, 2013, at 12:23 PM, Steven Bosscher wrote: >> On Tue, Sep 24, 2013 at 5:03 PM, Eric Botcazou wrote: This patch removes REG_DEAD and REG_UNUSED notes >>> DF framework is supposed to do it for you. >> Unfortunately LRA uses its own DF framew

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-25 Thread Vladimir Makarov
On 09/24/2013 10:40 AM, Yvan Roux wrote: > Hi, > > This patch removes REG_DEAD and REG_UNUSED notes in update_inc_notes, > as it is what the function is supposed to do (see the comments) and as > keeping these notes produce some failures, at least on ARM. > > Thanks, > Yvan > > 2013-09-24 Yvan Rou

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-24 Thread Mike Stump
On Sep 24, 2013, at 12:23 PM, Steven Bosscher wrote: > On Tue, Sep 24, 2013 at 5:03 PM, Eric Botcazou wrote: >>> This patch removes REG_DEAD and REG_UNUSED notes >> DF framework is supposed to do it for you. > Unfortunately LRA uses its own DF framework. Is that a bug, or a feature?

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-24 Thread Steven Bosscher
On Tue, Sep 24, 2013 at 5:03 PM, Eric Botcazou wrote: >> This patch removes REG_DEAD and REG_UNUSED notes in update_inc_notes, >> as it is what the function is supposed to do (see the comments) and as >> keeping these notes produce some failures, at least on ARM. > > The description is too terse.

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-24 Thread Yvan Roux
> The description is too terse. In the RTL middle-end, you shouldn't have to > manually deal with the REG_DEAD and REG_UNUSED notes (unlike REG_EQUAL and > REG_EQUIV notes), as the DF framework is supposed to do it for you. Sorry, for that. The description of the LRA function update_inc_notes ex

Re: [PATCH, LRA] Remove REG_DEAD and REG_UNUSED notes.

2013-09-24 Thread Eric Botcazou
> This patch removes REG_DEAD and REG_UNUSED notes in update_inc_notes, > as it is what the function is supposed to do (see the comments) and as > keeping these notes produce some failures, at least on ARM. The description is too terse. In the RTL middle-end, you shouldn't have to manually deal