On Tue, Feb 05, 2013 at 04:27:15PM +0100, Jan Hubicka wrote:
> I managed to get an accidental commit on those two testcases without an
> ChangeLog entry.
> I just commit the missing ChangeLog and the following patch that should make
> them pass.
>
> just in a case you wonder what are the tests about. I disabled iteration on
> early inliner
> that makes it to mis some cases of indirect inlining that are now handled by
> later
> inliner. The tree-ssa/inline-?.C testcases tests that early inliner with
> large enough
> iteration counts still does the optimization, while ipa/inline-?.C are the
> same
> testcases with no iteration and test that late inline picks the optimization
> too.
>
> The motivation for the change was early inliner doing funny things on
> non-trivial recursion
> especially with early-inlining-insns bumped up.
Doesn't seem to be enough.
gcc/testsuite/g++.dg/ipa/inline-3.C
has
/* { dg-final { cleanup-tree-dump "inline" } } */
while supposedly it should have:
/* { dg-final { cleanup-ipa-dump "inline" } } */
Also,
gcc/testsuite/gcc.dg/tree-ssa/inline-3.c
is now failing, supposedly it should have --param added ?
Jakub
