[PATCH] [LLD] COFF: Make link order compatible with MSVC link.exe.

Rui Ueyama ruiu at google.com
Tue Jun 23 16:15:51 PDT 2015


Hi pcc,

Previously, we added files in directive sections to the symbol
table as we read the sections, so the link order was depth-first.
That's not compatible with MSVC link.exe nor the old LLD.

This patch is to queue files so that new files are added to the
end of the queue and processed last. Now addFile() doesn't parse
files nor resolve symbols. You need to call run() to process
queued files.

http://reviews.llvm.org/D10672

Files:
  COFF/Driver.cpp
  COFF/Driver.h
  COFF/SymbolTable.cpp
  COFF/SymbolTable.h
  COFF/Writer.cpp
  test/COFF/order.test

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10672.28300.patch
Type: text/x-patch
Size: 12063 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150623/652f71f6/attachment.bin>


More information about the llvm-commits mailing list