Re: [PR64164] drop copyrename, integrate into expand

[Jeff Law <law@redhat.com>]

On 08/19/2015 06:00 PM, Alexandre Oliva wrote:
> On Aug 19, 2015, Alexandre Oliva <aoliva@redhat.com> wrote:
>
>> I'm having some difficulty getting access to an ia64 box ATM, and for
>> ada bootstraps, a cross won't do, so...  if you still have that build
>> tree around, any chance you could recompile par.o with both stage1 and
>> stage2, with -fdump-rtl-expand-details, and email me the compiler dump
>> files?
>
> Thanks!
>
> In the mean time, I have been able to duplicate the problem myself.  As
> you say, it is triggered by -gtoggle.  However, it has nothing
> whatsoever to do with the recent patches I installed.  At most they
> expose some latent problem in the scheduler.
>
> I have verified in the expand dumps that both the gimple and the rtl
> representation in the relevant parts of the code are identical, except
> for the presence of debug stmts and insns.  Indeed, compiling with
> -fno-schedule-insns{,2}, no differences arise.
We did a couple fixes to this code earlier this year.  Presumably 
there's something still subtly wrong in there that your changes are 
exposing.

See Maxim's changes from Feb.

You might also look and see if any of those insns have SCHED_GROUP_P set.

Jeff