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