[PATCH] D156618: [IR] Fix a memory leak if Function::dropAllReferences() is followed by setHungoffOperand

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

In D156618#4636483 <https://reviews.llvm.org/D156618#4636483>, @taolq wrote:

> In D156618#4632192 <https://reviews.llvm.org/D156618#4632192>, @dexonsmith wrote:
>
>> In D156618#4632179 <https://reviews.llvm.org/D156618#4632179>, @dexonsmith wrote:
>>
>>> Seems unfortunate to add calls to `ConstantPointerNull::get()` when all of the callers are about to call `delete`.
>>>
>>> Can we instead somehow enforce that `delete` is always called right after? Perhaps add an assertion that this is so?
>>
>> Or, looking back at the motivation (calling `deleteBody()`), can we have two entry points?
>>
>> - One prepares for deletion (99% of callers), skips `CPN::get()`
>> - One makes it a prototype (deleteBody), calls `CPN::get()`
>
> Thanks for the comment. I think it is a little redundant. Since we already has dector to release the memory, another entry points seems redundant.
> So I'd like to reserve the only prototype version.

What is "dector"?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156618/new/

https://reviews.llvm.org/D156618