[libc-commits] [PATCH] D73337: Fix missing dependency in LibcUnitTest

[Siva Chandra via Phabricator via libc-commits]

sivachandra added a comment.

In D73337#1840618 <https://reviews.llvm.org/D73337#1840618>, @abrachet wrote:

> I think this patch is still correct to add this, though. Even if the unittest itself will be linked to Support and also linked against LibcUnitTest and even if LibcUnitTest will never be used outside of that context, it would still make sense for LibcUnitTest to link against everything it needs. At worst this can't hurt.


If you want to keep it here, then you should remove it from the list of the test exe's link libraries. I have verified that such a scheme works. So, with that change included, this change is alright with me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73337