[PATCH] D75291: Rename "llvm-gsym" and "llvm-gsymutil" to "gsymutil".

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 23:26:39 PST 2020


clayborg updated this revision to Diff 247168.
clayborg added a comment.

Renamed back to llvm-gsymutil for everything. So this patch now is renaming from llvm-gsym to llvm-gsymutil for the tool directory.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75291

Files:
  llvm/tools/llvm-gsym/CMakeLists.txt
  llvm/tools/llvm-gsym/llvm-gsymutil.cpp
  llvm/tools/llvm-gsymutil/CMakeLists.txt
  llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
  llvm/utils/gn/secondary/llvm/test/BUILD.gn
  llvm/utils/gn/secondary/llvm/tools/llvm-gsym/BUILD.gn
  llvm/utils/gn/secondary/llvm/tools/llvm-gsymutil/BUILD.gn


Index: llvm/utils/gn/secondary/llvm/tools/llvm-gsymutil/BUILD.gn
===================================================================
--- llvm/utils/gn/secondary/llvm/tools/llvm-gsymutil/BUILD.gn
+++ llvm/utils/gn/secondary/llvm/tools/llvm-gsymutil/BUILD.gn
@@ -1,9 +1,7 @@
 executable("llvm-gsymutil") {
   deps = [
-    "//llvm/lib/CodeGen/AsmPrinter",
     "//llvm/lib/DebugInfo/DWARF",
     "//llvm/lib/DebugInfo/GSYM",
-    "//llvm/lib/MC",
     "//llvm/lib/Object",
     "//llvm/lib/Support",
     "//llvm/lib/Target",
Index: llvm/utils/gn/secondary/llvm/test/BUILD.gn
===================================================================
--- llvm/utils/gn/secondary/llvm/test/BUILD.gn
+++ llvm/utils/gn/secondary/llvm/test/BUILD.gn
@@ -216,7 +216,7 @@
     "//llvm/tools/llvm-elfabi",
     "//llvm/tools/llvm-exegesis",
     "//llvm/tools/llvm-extract",
-    "//llvm/tools/llvm-gsym:llvm-gsymutil",
+    "//llvm/tools/llvm-gsymutil:llvm-gsymutil"
     "//llvm/tools/llvm-ifs",
     "//llvm/tools/llvm-isel-fuzzer",
     "//llvm/tools/llvm-jitlink",
Index: llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
===================================================================
--- llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
+++ llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
@@ -1,4 +1,4 @@
-//===-- llvm-gsymutil.cpp - GSYM dumping and creation utility for llvm ----===//
+//===-- gsymutil.cpp - GSYM dumping and creation utility for llvm ---------===//
 //
 //                     The LLVM Compiler Infrastructure
 //
@@ -350,7 +350,8 @@
   error(Filename, errorToErrorCode(BinOrErr.takeError()));
 
   if (auto *Obj = dyn_cast<ObjectFile>(BinOrErr->get())) {
-    auto ArchName = Obj->makeTriple().getArchName();
+    Triple ObjTriple = Obj->makeTriple();
+    StringRef ArchName = ObjTriple.getArchName();
     outs() << "Output file (" << ArchName << "): " << OutFile << "\n";
     if (auto Err = handleObjectFile(*Obj, OutFile.c_str()))
       return Err;
@@ -428,10 +429,7 @@
   PrettyStackTraceProgram X(argc, argv);
   llvm_shutdown_obj Y; // Call llvm_shutdown() on exit.
 
-  llvm::InitializeAllTargetInfos();
-  llvm::InitializeAllTargetMCs();
   llvm::InitializeAllTargets();
-  llvm::InitializeAllAsmPrinters();
 
   const char *Overview =
       "A tool for dumping, searching and creating GSYM files.\n\n"
Index: llvm/tools/llvm-gsymutil/CMakeLists.txt
===================================================================
--- llvm/tools/llvm-gsymutil/CMakeLists.txt
+++ llvm/tools/llvm-gsymutil/CMakeLists.txt
@@ -2,10 +2,6 @@
   ${LLVM_TARGETS_TO_BUILD}
   DebugInfoDWARF
   DebugInfoGSYM
-  AsmPrinter
-  AllTargetsDescs
-  AllTargetsInfos
-  MC
   Object
   Support
   Target
@@ -14,6 +10,4 @@
 add_llvm_tool(llvm-gsymutil
   llvm-gsymutil.cpp
 
-  DEPENDS
-  intrinsics_gen
   )


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75291.247168.patch
Type: text/x-patch
Size: 2788 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200228/4d45fab9/attachment.bin>


More information about the llvm-commits mailing list