[PATCH] D109294: [Inline] Remove dead users before checking if function has one use (PR51667)

Mircea Trofin via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 11 11:20:12 PDT 2021


An analysis could count uses, for example; or could enumerate constants in
a module and refine some calculation from that set, but
removeDeadUsersOfConstant may delete some such constants.

(what's a "VH-like thing"?)

On Fri, Sep 10, 2021 at 1:53 PM Roman Lebedev via Phabricator <
reviews at reviews.llvm.org> wrote:

> lebedev.ri added a comment.
>
> Passing-by remark: i really don't see why would any analysis depend on
> *dead* constant use.
> If they do, they sound really fragile. The obvious fix for them is to
> actually signal that;
> i believe there was some VH-like thing for that?
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D109294/new/
>
> https://reviews.llvm.org/D109294
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210911/da9a3fec/attachment.html>


More information about the llvm-commits mailing list