[LLD] [COFF] Fix def file exporting of symbols containing periods

[mstorsjo]

Summary

This fixes an accidental breakage of exporting symbols using def files, when the symbol name contains a period, since commit 0ca06f7950e5, mixing up a symbol name containing a period with the case of exporting a symbol as a forward to another dll.

[mati865]

Confirming that this patch fixes linking MinGW Rust with LLD.

[rnk]

lgtm

lld/COFF/Driver.cpp

if (m.MinorOSVersion)
     config->minorOSVersion = m.MinorOSVersion;
 
   for (COFFShortExport e1 : m.Exports) {
     Export e2;
     // 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. This is supported by both MS and GNU linkers.
-    if (e1.ExtName != e1.Name && StringRef(e1.Name).contains('.')) {
+    if (!e1.ExtName.empty() && e1.ExtName != e1.Name &&
+        StringRef(e1.Name).contains('.')) {
       e2.name = saver.save(e1.ExtName);
       e2.forwardTo = saver.save(e1.Name);
       config->exports.push_back(e2);
       continue;
     }
     e2.name = saver.save(e1.Name);
     e2.extName = saver.save(e1.ExtName);
     e2.ordinal = e1.Ordinal;

lld/test/COFF/export-tricky-names.s

+# REQUIRES: x86
+
+# RUN: echo -e "EXPORTS\n  foo.bar" > %t.def
+
+# RUN: llvm-mc -filetype=obj -triple=x86_64-windows-msvc -defsym drectve=0 %s -o %t.obj
+# RUN: lld-link -entry:dllmain -dll -def:%t.def %t.obj -out:%t.1.dll
+# RUN: llvm-readobj --coff-exports %t.1.dll | FileCheck %s
+
+# RUN: lld-link -entry:dllmain -dll %t.obj -out:%t.2.dll -export:foo.bar
+# RUN: llvm-readobj --coff-exports %t.2.dll | FileCheck %s
+
+# RUN: llvm-mc -filetype=obj -triple=x86_64-windows-msvc -defsym drectve=1 %s -o %t.drectve.obj
+# RUN: lld-link -entry:dllmain -dll %t.drectve.obj -out:%t.3.dll
+# RUN: llvm-readobj --coff-exports %t.3.dll | FileCheck %s
+
+# CHECK: Name: foo.bar
+
+        .text
+        .globl  dllmain
+        .globl  foo.bar
+dllmain:
+        ret
+foo.bar:
+        ret
+
+.if drectve==1
+        .section .drectve
+        .ascii "-export:foo.bar"
+.endif