[PATCH] D114889: [GlobalOpt] Simplify CleanupConstantGlobalUsers()

Thu Dec 2 00:22:18 PST 2021

uabelho added a comment.

In D114889#3166181 <https://reviews.llvm.org/D114889#3166181>, @nikic wrote:

> In D114889#3166119 <https://reviews.llvm.org/D114889#3166119>, @uabelho wrote:
>
>> It looks like the issue described in
>>  https://bugs.llvm.org/show_bug.cgi?id=39751
>> stopped happening with this patch.
>
> Thanks for the pointer. I committed the test case in https://github.com/llvm/llvm-project/commit/bc61e5e90b8db92aa5464d0565c8993b776df54d -- just need to remember that the issue can be closed once that's possible again :)

Great, thanks!
Yes it's starting to pile up now. I have a list of things to do once it opens up somewhere, I can add PR39751 to my list and close it too.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114889