[PATCH] D46244: [LLD] [COFF] Allow specifying export forwarding in a def file

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 5 10:54:20 PDT 2018


mstorsjo added inline comments.


================
Comment at: COFF/Driver.cpp:631
     Export E2;
+    if (E1.ExtName != E1.Name && StringRef(E1.Name).contains('.')) {
+      E2.Name = E1.ExtName;
----------------
ruiu wrote:
> Can you add a comment?
Sure - I can add one like this:

In simple cases, only Name is set. Renamed exports are parsed and set as "ExtName = Name". If Name has the form "OtherDll.Func", it shouldn't be a normal exported function but a forward to another DLL instead.




Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D46244





More information about the llvm-commits mailing list