[PATCH] D53758: Handle value uses wrapped in metadata for the use-list order

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 10:28:52 PDT 2020


dstenb added a comment.

In D53758#2336719 <https://reviews.llvm.org/D53758#2336719>, @dexonsmith wrote:

> I totally lost track of this :/.
>
> The patch looks correct, thanks very much for tracking this down. This LGTM if you add an explanation to the testcase of exactly what it’s testing (I worry it’s not obvious for someone trying to update the test later).

Me too! Thanks a lot for reviewing this!

I have added an explanation to the test case.


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

https://reviews.llvm.org/D53758



More information about the llvm-commits mailing list