[PATCH] D99169: [DebugInfo] Replace debug uses in replaceUsesOutsideBlock

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 15 06:07:05 PDT 2021


Orlando added a comment.

In D99169#2683001 <https://reviews.llvm.org/D99169#2683001>, @Orlando wrote:

> In D99169#2678746 <https://reviews.llvm.org/D99169#2678746>, @djtodoro wrote:
>
>> Thanks! LGTM like this...
>>
>> (I vote for this simple code change rather than introducing this overloading, but please wait for couple of more days, so we can see if someone else has a different opinion.)
>
> Thanks @djtodoro. Sure, I will leave this up a few days before landing.

I came to do this just now but noticed that this patch isn't "Accepted". @djtodoro is this intentional? I'm happy to wait for another reviewer if so but I just wanted to check.


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

https://reviews.llvm.org/D99169



More information about the llvm-commits mailing list