[COFF] added test for thinlto

[inglorion]

Summary

Creates bitcode files suitable for use with ThinLTO, then checks that the linker can build an executable from them.

[mehdi_amini]

Maybe the testing is obvious to someone use to COFF, in which case I'm just out-of-place :)
Otherwise I'd rather have comment explaining what every check is doing.

test/COFF/thinlto.ll

10

is .obj the standard extension for object on Windows? (and .lib for archive?)

12

I don't know what is the /include directive doing? Is it preventing dead-stripping somehow? Or affecting the symbol resolution otherwise?

35

I don't know enough about COFF to know if what you're testing is relevant here. Especially I have no idea why you're checking the HEADERS (or what you're checking).

107

Why do we check this exact sequence of assembly? Overall I don't know what this test is enforcing / verifying right now.

[pcc]

For this test I wouldn't expect to see anything more than a basic test that uses temporaries to verify that importing is working. See http://llvm-cs.pcc.me.uk/tools/lld/test/ELF/lto/thinlto.ll for example.

[pcc]

See also r295901 which changes the ELF test to be closer to what I'd expect this test to be doing.

[inglorion]

@pcc: How is this instead?

[inglorion]

test/COFF/thinlto.ll

10

Yes.

12

Right. It instructs the linker to retain that symbol, regardless of what it would otherwise do.

107

This was essentially performing the same checks as lto.ll. However, per @pcc's suggestion, I've changed the test to be much simpler and not actually look at the disassembly.

[pcc]

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.

[mehdi_amini]

test/COFF/thinlto.ll

107

Thanks, looks good!
(and I agree with the idea of using llvm-nm where possible/appropriate)

[inglorion]

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.

[pcc]

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

[mehdi_amini]

In 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 :)

[inglorion]

use llvm-nm instead of looking at the bitcode

[mehdi_amini]

test/COFF/thinlto.ll

8

assuming llvm-nm on COFF looks like ELF, what you need to check is not that main is present, but that foo isn't.

[pcc]

test/COFF/thinlto.ll

8

; CHECK-NOT: U foo?

[inglorion]

check for absence of undefined foo, instead of presence of main

[pcc]

LGTM

test/COFF/thinlto.ll

+; RUN: rm -fr %T/thinlto
+; RUN: mkdir %T/thinlto
+; RUN: opt -thinlto-bc -o %T/thinlto/main.obj %s
+; RUN: opt -thinlto-bc -o %T/thinlto/foo.obj %S/Inputs/lto-dep.ll
+; RUN: lld-link /lldsavetemps /out:%T/thinlto/main.exe /entry:main /subsystem:console %T/thinlto/main.obj %T/thinlto/foo.obj
+; RUN: llvm-nm %T/thinlto/main.exe.lto.obj | FileCheck %s
+
+; CHECK: T main
+
+target datalayout = "e-m:w-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-pc-windows-msvc"
+
+define i32 @main() {
+  call void @foo()
+  ret i32 0
+}
+
+declare void @foo()