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  <mjam...@suse.cz>
> 
>       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

Reply via email to