@Honza: PING^1
On 9/18/19 12:14 PM, Martin Liška wrote:
> On 9/11/19 1:38 PM, Martin Liška wrote:
>> The inline_clone manipulation happens in cgraph_node::find_replacement where
>> we manipulate the clone_of.
>
> I fixed that but there's a similar situation which goes other way around:
>
> cgraph_node *
> cgraph_node::get_create (tree decl)
> {
> cgraph_node *first_clone = cgraph_node::get (decl);
>
> if (first_clone && !first_clone->global.inlined_to)
> return first_clone;
>
> cgraph_node *node = cgraph_node::create (decl);
> if (first_clone)
> {
> first_clone->clone_of = node;
>
> Here we come up with a new parent and this->clone_of is set to the parent.
> We ought to come cgraph_node::order here, but I don't like.
> Right now cgraph_node::order is a way how one can identify a node in IPA
> dumps.
>
> The patch is breaking that. I'm not sure we want the patch right now.
> Martin
>