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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc 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.


Do you mean that `main.exe.lto.obj` only contains a definition of main? That is what I'd expect to see if foo was imported into main and inlined.


https://reviews.llvm.org/D30277