[PATCH] D61699: [lld-link] initialize targets and asmparsers before invoking lib
Bob Haarman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 15:08:47 PDT 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rLLD360295: [lld-link] initialize targets and asmparsers before invoking lib (authored by inglorion, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D61699?vs=198728&id=198735#toc
Repository:
rLLD LLVM Linker
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61699/new/
https://reviews.llvm.org/D61699
Files:
COFF/Driver.cpp
test/COFF/lib-module-asm.ll
Index: COFF/Driver.cpp
===================================================================
--- COFF/Driver.cpp
+++ COFF/Driver.cpp
@@ -975,6 +975,13 @@
}
void LinkerDriver::link(ArrayRef<const char *> ArgsArr) {
+ // Needed for LTO.
+ InitializeAllTargetInfos();
+ InitializeAllTargets();
+ InitializeAllTargetMCs();
+ InitializeAllAsmParsers();
+ InitializeAllAsmPrinters();
+
// If the first command line argument is "/lib", link.exe acts like lib.exe.
// We call our own implementation of lib.exe that understands bitcode files.
if (ArgsArr.size() > 1 && StringRef(ArgsArr[1]).equals_lower("/lib")) {
@@ -983,13 +990,6 @@
return;
}
- // Needed for LTO.
- InitializeAllTargetInfos();
- InitializeAllTargets();
- InitializeAllTargetMCs();
- InitializeAllAsmParsers();
- InitializeAllAsmPrinters();
-
// Parse command line options.
ArgParser Parser;
opt::InputArgList Args = Parser.parseLINK(ArgsArr);
Index: test/COFF/lib-module-asm.ll
===================================================================
--- test/COFF/lib-module-asm.ll
+++ test/COFF/lib-module-asm.ll
@@ -0,0 +1,21 @@
+; PR41803: llvm-link /lib on object with module asm crashes
+; RUN: rm -f %t.lib
+; RUN: llvm-as -o %t.obj %s
+; RUN: lld-link /lib /out:%t.lib %t.obj
+; RUN: llvm-nm -M %t.lib | FileCheck %s
+
+target datalayout = "e-m:x-p:32:32-i64:64-f80:32-n8:16:32-a:0:32-S32"
+target triple = "i386-pc-windows-msvc19.11.0"
+
+module asm ".global global_asm_sym"
+module asm "global_asm_sym:"
+module asm "local_asm_sym:"
+module asm ".long undef_asm_sym"
+
+; CHECK: Archive map
+; CHECK-NEXT: global_asm_sym in {{.*}}lib-module-asm.ll.tmp.obj
+
+; CHECK: lib-module-asm.ll.tmp.obj:{{$}}
+; CHECK-NEXT: T global_asm_sym
+; CHECK-NEXT: t local_asm_sym
+; CHECK-NEXT: U undef_asm_sym
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61699.198735.patch
Type: text/x-patch
Size: 1826 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190508/0a012289/attachment.bin>
More information about the llvm-commits
mailing list