On Fri, Nov 15, 2013 at 11:02 AM, Jan Hubicka <hubi...@ucw.cz> wrote:

>> >> >     * lto-streamer-in.c (input function): Call cgraph_create_node if
>> >> >     cgraph_get_node failed.
>> >> >
>> >> > Tested with lto-profiledbootstrap on x86_64-pc-linux-gnu, regression
>> >> > tested also with -m32 [1].
>> >> >
>> >> > OK for mainline?
>> >
>> > OK (though it is a bit ugly - but lto streaming is very much a special 
>> > case),
>> > thanks!
>>
>> Why would the cgraph node be not present when we input the body!?
>> Isn't there a bug elsewhere??
>>
>> IMHO this just papers over an issue.
>
> Hmm, right, I misread it to be lto-cgraph.c instead (where it would make 
> sense,
> since we may read nodes previously constructed earlier by decl sharing).
>
> I will look into this.  We should to have nodes present here.

Please also test your change with lto-profiledbootstrap. The problem
didn't trigger with lto-bootstrap.

Uros.

Reply via email to