[PATCH] D123185: [LLD][COFF] Fix TypeServerSource matcher with more than one collision

Sylvain Audi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 6 08:56:25 PDT 2022


saudi accepted this revision.
saudi added a comment.

LGTM
I was curious whether it would be good to remove the notion of GUID validity from the tests, since we now have support for collision whatever the GUIDs are. 
I just noticed it, but it was already the case in D122372 <https://reviews.llvm.org/D122372>, so it might be for a separate patch if we want that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123185



More information about the llvm-commits mailing list