[lld] Don't create hints-section if Hint/Name Table is empty
Fixes assert in addLinkerModuleCoffGroup() when using by-ordinal imports
only.
Patch by Stefan Schmidt.
Differential revision: https://reviews.llvm.org/D68352
git-svn-id: https://llvm.org/svn/llvm-project/lld/trunk@374140 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/COFF/Writer.cpp b/COFF/Writer.cpp
index a98af43..8387361 100644
--- a/COFF/Writer.cpp
+++ b/COFF/Writer.cpp
@@ -743,7 +743,8 @@
add(".idata$2", idata.dirs);
add(".idata$4", idata.lookups);
add(".idata$5", idata.addresses);
- add(".idata$6", idata.hints);
+ if (!idata.hints.empty())
+ add(".idata$6", idata.hints);
add(".idata$7", idata.dllNames);
}
diff --git a/test/COFF/Inputs/ordinal-only-implib.def b/test/COFF/Inputs/ordinal-only-implib.def
new file mode 100644
index 0000000..d913ef4
--- /dev/null
+++ b/test/COFF/Inputs/ordinal-only-implib.def
@@ -0,0 +1,3 @@
+LIBRARY test.dll
+EXPORTS
+ByOrdinalFunction @ 1 NONAME
diff --git a/test/COFF/imports-ordinal-only.s b/test/COFF/imports-ordinal-only.s
new file mode 100644
index 0000000..d4f5c44
--- /dev/null
+++ b/test/COFF/imports-ordinal-only.s
@@ -0,0 +1,18 @@
+# REQUIRES: x86
+#
+# RUN: llvm-dlltool -k -m i386 --input-def %p/Inputs/ordinal-only-implib.def --output-lib %t-implib.a
+# RUN: llvm-mc -triple=i386-pc-win32 %s -filetype=obj -o %t.obj
+# RUN: lld-link -out:%t.exe -entry:main -subsystem:console -safeseh:no -debug %t.obj %t-implib.a
+# RUN: llvm-objdump -private-headers %t.exe | FileCheck --match-full-lines %s
+
+.text
+.global _main
+_main:
+call _ByOrdinalFunction
+ret
+
+# CHECK: The Import Tables:
+# CHECK: DLL Name: test.dll
+# CHECK-NEXT: Hint/Ord Name
+# CHECK-NEXT: 1
+# CHECK-EMPTY:
