[PATCH] D78894: [ms] Add new option to llvm-lib: /llvmemptylib

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 12:55:51 PDT 2020


thakis added a subscriber: mstorsjo.
thakis added a comment.

When creating a dll, we currently create an empty .lib file (for similar reasons as in the patch description), while link.exe only creates a .lib file if there actually are exports. I wonder if we should change that to match link.exe by default and give lld its current behavior if this new flag here is passed? Not sure, maybe @mstorsjo has an opinion (but maybe he doesn't care about link.exe compat all that much) :)

Otherwise, this looks fine to me once it has a test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78894





More information about the llvm-commits mailing list