On Tue, Apr 17, 2018 at 08:55:09AM +0200, Martin Jambor wrote:
> Hi,
>
> my fix from last week caused an ICE described in PR 85421 because I did
> not read my own comment and forgot that after calls to
> redirect_callee_duplicating_thunks, one must finish the work by calling
> expand_all_artificial_thunks, otherwise we end up with unanalyzed thunks
> and hit some assert sooner or later, like we did in this bug.
>
> Fixed with the patch below, bootstrapped, LTO-bootstrapped and tested on
> x86_64-linux. OK for trunk?
>
> I'm sorry for introducing an ICE so late in stage4,
>
> Martin
>
>
> 2018-04-17 Martin Jambor <[email protected]>
>
> PR ipa/85421
> * ipa-cp.c (create_specialized_node): Call
> expand_all_artificial_thunks if necessary.
> * testsuite/g++.dg/ipa/pr85421.C: New test.
Ok for trunk.
No testsuite/ prefix for files going into testsuite/ChangeLog.
Jakub
