[PATCH] D89775: [lld][ELF][test] Add additional test coverage for LTO

Wed Oct 21 07:48:29 PDT 2020

jhenderson planned changes to this revision.
jhenderson added a comment.

I'll see if I can merge common-mixed1.ll and common-mixed2.ll and inline the input files. That should allow me to avoid renaming common4.ll too.

================
Comment at: lld/test/ELF/lto/resolution.ll:15
 ; CHECK-NEXT: Offset:
-; CHECK-NEXT: Size: 4
+; CHECK-NEXT: Size: 8
 ; CHECK-NEXT: Link: 0
----------------
This test is failing for reasons I've yet to discover. I thought it passed previously, but it's possible it didn't. Currently, it fails because both versions of b are present in the contents of .data, which clearly isn't correct (only one b appears in the symbol table).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89775