Possible Bugs in cgraphunit.c

[Nicholas Krause]

On 12/5/19 7:08 AM, Martin Liška wrote:
> On 12/5/19 9:00 AM, Nicholas Krause wrote:
>> Greetings,
>> Seems that the extend_trucks return values are not returned when 
>> called in both,
>> cnode::assemble_thunks_and_aliases and cnode::create_wrapper. I'm not 
>> sure
>> if this is a set of edge case bugs or there was a reason for this. 
>> Seems not as its
>> checked in the third function caller in the file, cgraph_node::analyze.
>
> Hello.
>
> You are right that the return value of expand_thunk is not checked 
> (except one place).
> The code is quite old, so I guess it's not causing issues.
>
> Martin
>
Indeed or the edge cases are almost never hit or not at all. So the 
question is should
we just change the function to return void as its always successful or 
fix up the callers?

Nick
>>
>> Not sure if my assumptions are correct so I'm asking if there isn't 
>> another reason
>> for this as the code seems to have at least directly no reason for it,
>>
>> Nick
>