Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-28 Thread Vladimir Makarov
On 12-10-27 6:07 PM, Steven Bosscher wrote: On Sat, Oct 27, 2012 at 11:49 PM, Steven Bosscher wrote: So I'm going to revert this patch. Or actually better, this lighter-weight patch. Will commit after the usual testing. I returned to this problem several times and every time I failed to use

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-27 Thread Steven Bosscher
On Sat, Oct 27, 2012 at 11:49 PM, Steven Bosscher wrote: > So I'm going to revert this patch. Or actually better, this lighter-weight patch. Will commit after the usual testing. * ira.c (ira): Remove DF_LIVE if the problem is in the stack. (do_reload): Add it back at the end. Inde

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-27 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 11:12 PM, Vladimir Makarov wrote: > On 12-10-13 11:37 AM, Steven Bosscher wrote: >>> LIVE is not used on purpose. I added LIVE usage to the old RA about 10 >>> years ago (it was before DF-infrastructure). Everything was ok until, >>> somebody reported compiler crash on sem

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-14 Thread Steven Bosscher
On Sun, Oct 14, 2012 at 7:19 PM, Vladimir Makarov wrote: > Thanks, Steven. IRA part is ok for me to commit. Thanks, I've committed this as trunk r192440. I'm aware I'm on the hook for fixing any fall-out :-) Ciao! Steven

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-14 Thread Vladimir Makarov
On 12-10-14 6:16 AM, Steven Bosscher wrote: On Sat, Oct 13, 2012 at 11:12 PM, Vladimir Makarov wrote: Ok for the idea. If we have a problem later, we could fix it. I'll look at the next version of the patch when you send it to give your the final approval. Great, thanks! Here is the updated

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-14 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 11:12 PM, Vladimir Makarov wrote: > Ok for the idea. If we have a problem later, we could fix it. I'll look at > the next version of the patch when you send it to give your the final > approval. Great, thanks! Here is the updated patch, tested in the same way as the pre

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 11:05 PM, Eric Botcazou wrote: >> I think it would be a good idea to keep things unchanged at -O1. For >> that, the patch needs a few minor modifications (remove calls to >> df_live_add_problem and make some code to update DF_LIVE_{IN,OUT} >> conditional). I can prepare an u

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Vladimir Makarov
On 12-10-13 11:37 AM, Steven Bosscher wrote: On Sat, Oct 13, 2012 at 5:12 PM, Vladimir Makarov wrote: On 12-10-13 9:40 AM, Steven Bosscher wrote: Hello, This fixes the long-standing enhancement request to use DF_LIVE in IRA. To quote the first comment in the PR: IRA should be using the DF-LIV

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Eric Botcazou
> I think it would be a good idea to keep things unchanged at -O1. For > that, the patch needs a few minor modifications (remove calls to > df_live_add_problem and make some code to update DF_LIVE_{IN,OUT} > conditional). I can prepare an updated patch for that, if you think > that's best. That wa

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 10:38 PM, Eric Botcazou wrote: > So, in the end, does the patch enable DF_LIVE at -O1 or not? There seems to > be a contradiction between the subject and the body of the message. If yes, > perhaps an acceptable compromise would be to keep things unchanged at -O1. Eh, rig

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Eric Botcazou
> IRA should be using the DF-LIVE sets, which are smaller than the DF-LR > sets when they are available (typically at O2 and above). The proper > sets can be conveniently accessed using the df_get_live_[in,out] > functions which use DF-LIVE if it is available and fall back to DF-LR > if it is not.

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 5:12 PM, Vladimir Makarov wrote: > On 12-10-13 9:40 AM, Steven Bosscher wrote: >> >> Hello, >> >> This fixes the long-standing enhancement request to use DF_LIVE in >> IRA. To quote the first comment in the PR: >> >> IRA should be using the DF-LIVE sets, which are smaller th

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Vladimir Makarov
On 12-10-13 9:40 AM, Steven Bosscher wrote: Hello, This fixes the long-standing enhancement request to use DF_LIVE in IRA. To quote the first comment in the PR: IRA should be using the DF-LIVE sets, which are smaller than the DF-LR sets when they are available (typically at O2 and above). The p

[PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-13 Thread Steven Bosscher
Hello, This fixes the long-standing enhancement request to use DF_LIVE in IRA. To quote the first comment in the PR: IRA should be using the DF-LIVE sets, which are smaller than the DF-LR sets when they are available (typically at O2 and above). The proper sets can be conveniently accessed using