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

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 11:53:17 PST 2017


mehdi_amini added a comment.

In https://reviews.llvm.org/D30277#684874, @inglorion wrote:

> > 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.
>
> That was my original plan, but it looks like llvm-nm only tells us about main. Is that enough? Is there a way to get it to give us more information? I wrote the test the way I wrote it because I felt just looking at the output from llvm-nm doesn't really tell us if ThinLTO is working as intended.


If there is no undefined reference to foo from main, then it means it worked :)


https://reviews.llvm.org/D30277





More information about the llvm-commits mailing list