[PATCH] D118575: [llvm-libtool-darwin] Fix crash with bitcode asm module

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 4 10:55:55 PST 2022


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGdbed14d215fe: [llvm-libtool-darwin] Fix crash with bitcode asm module (authored by keith).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118575

Files:
  llvm/test/tools/llvm-libtool-darwin/Inputs/arm64-asm.ll
  llvm/test/tools/llvm-libtool-darwin/Inputs/x86_64-asm.ll
  llvm/test/tools/llvm-libtool-darwin/arm64-asm.test
  llvm/test/tools/llvm-libtool-darwin/x86_64-asm.test
  llvm/tools/llvm-libtool-darwin/CMakeLists.txt
  llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp


Index: llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp
===================================================================
--- llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp
+++ llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp
@@ -21,6 +21,7 @@
 #include "llvm/Support/CommandLine.h"
 #include "llvm/Support/InitLLVM.h"
 #include "llvm/Support/LineIterator.h"
+#include "llvm/Support/TargetSelect.h"
 #include "llvm/Support/VirtualFileSystem.h"
 #include "llvm/Support/WithColor.h"
 #include "llvm/Support/raw_ostream.h"
@@ -682,6 +683,10 @@
   if (VersionOption)
     cl::PrintVersionMessage();
 
+  llvm::InitializeAllTargetInfos();
+  llvm::InitializeAllTargetMCs();
+  llvm::InitializeAllAsmParsers();
+
   Config C = *ConfigOrErr;
   switch (LibraryOperation) {
   case Operation::None:
Index: llvm/tools/llvm-libtool-darwin/CMakeLists.txt
===================================================================
--- llvm/tools/llvm-libtool-darwin/CMakeLists.txt
+++ llvm/tools/llvm-libtool-darwin/CMakeLists.txt
@@ -4,6 +4,7 @@
   Object
   Support
   TextAPI
+  ${LLVM_TARGETS_TO_BUILD}
   )
 
 add_llvm_tool(llvm-libtool-darwin
Index: llvm/test/tools/llvm-libtool-darwin/x86_64-asm.test
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-libtool-darwin/x86_64-asm.test
@@ -0,0 +1,9 @@
+# REQUIRES: x86-registered-target
+## This tests that archives are correctly created when the llvm
+## has native assembly info
+
+# RUN: llvm-as %p/Inputs/x86_64-asm.ll -o %t-x86_64-asm.bc
+# RUN: llvm-libtool-darwin -static -o %t-x86_64.lib %t-x86_64-asm.bc
+# RUN: llvm-nm %t-x86_64.lib | FileCheck %s
+
+# CHECK: T somesymbol
Index: llvm/test/tools/llvm-libtool-darwin/arm64-asm.test
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-libtool-darwin/arm64-asm.test
@@ -0,0 +1,9 @@
+# REQUIRES: aarch64-registered-target
+## This tests that archives are correctly created when the llvm
+## has native assembly info
+
+# RUN: llvm-as %p/Inputs/arm64-asm.ll -o %t-arm64-asm.bc
+# RUN: llvm-libtool-darwin -static -o %t-arm64.lib %t-arm64-asm.bc
+# RUN: llvm-nm %t-arm64.lib | FileCheck %s
+
+# CHECK: T somesymbol
Index: llvm/test/tools/llvm-libtool-darwin/Inputs/x86_64-asm.ll
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-libtool-darwin/Inputs/x86_64-asm.ll
@@ -0,0 +1,7 @@
+target triple = "x86_64-apple-macosx11.0.0"
+
+module asm ".desc ___crashreporter_info__, 0x10"
+
+define void @somesymbol() {
+  ret void
+}
Index: llvm/test/tools/llvm-libtool-darwin/Inputs/arm64-asm.ll
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-libtool-darwin/Inputs/arm64-asm.ll
@@ -0,0 +1,7 @@
+target triple = "arm64-apple-macosx11.0.0"
+
+module asm ".desc ___crashreporter_info__, 0x10"
+
+define void @somesymbol() {
+  ret void
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118575.406039.patch
Type: text/x-patch
Size: 2966 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220204/5a79bfb7/attachment.bin>


More information about the llvm-commits mailing list