[PATCH] D32317: [coff] for /msvclto, pass archive members with prevailing symbols first

Bob Haarman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 15:17:22 PDT 2017


inglorion created this revision.
Herald added a subscriber: mehdi_amini.

When using /msvclto, lld and MSVC's linker both do their own symbol resolution. This can cause them to select different archive members, which can result in undefined references. This change avoids that situation by extracting archive members that are selected by lld and passing those to link.exe before any archives, so that MSVC's uses those objects for symbol resolution instead of different archive members.


https://reviews.llvm.org/D32317

Files:
  COFF/Config.h
  COFF/Driver.cpp
  test/COFF/Inputs/msvclto-order-a.ll
  test/COFF/Inputs/msvclto-order-b.ll
  test/COFF/msvclto-order.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32317.96038.patch
Type: text/x-patch
Size: 3802 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170420/5ed2e64f/attachment.bin>


More information about the llvm-commits mailing list