Displaying 2 results from an estimated 2 matches for "addphysregcoalesce".
2014 Sep 10
4
[LLVMdev] Leaks in PBQPBuilderWithCoalescing::build ?
Hi Lang,
In PBQPBuilderWithCoalescing::build, around line 360, we have code looking like:
…
PBQP::Vector newCosts(g.getNodeCosts(node));
addPhysRegCoalesce(newCosts, pregOpt, cBenefit);
g.setNodeCosts(node, newCosts);
…
I suspect the leak occurs around the setNodeCosts method, and I have trouble understanding how it handles the case where the node already has costs.
It seems to me that:
- we make of copy of the node’s costs (probably becaus...
2014 Sep 10
4
[LLVMdev] Leaks in PBQPBuilderWithCoalescing::build ?
...andmaison at arm.com> wrote:
>
>> Hi Lang,
>>
>>
>>
>> In PBQPBuilderWithCoalescing::build, around line 360, we have code
>> looking like:
>>
>>
>>
>> …
>>
>> PBQP::Vector newCosts(g.getNodeCosts(node));
>>
>> addPhysRegCoalesce(newCosts, pregOpt, cBenefit);
>>
>> g.setNodeCosts(node, newCosts);
>>
>> …
>>
>>
>>
>> I suspect the leak occurs around the setNodeCosts method, and I have
>> trouble understanding how it handles the case where the node already has
>> cos...