[PATCH] D30277: [COFF] added test for thinlto

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 16:44:47 PST 2017


pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

Nit: I'd use llvm-nm to check the symbol tables of the object files so that we're checking the inputs that the linker actually reads, rather than a byproduct of LTO. Other than that, LGTM.


https://reviews.llvm.org/D30277





More information about the llvm-commits mailing list