[PATCH] D97682: [compiler-rt] PR#39514 Support versioned llvm-symbolizer binaries

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 18:29:33 PST 2021


This revision was automatically updated to reflect the committed changes.
Closed by commit rG3d039f65015f: [compiler-rt] PR#39514 Support versioned llvm-symbolizer binaries (authored by ahunt, committed by eugenis).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97682

Files:
  compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp


Index: compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
===================================================================
--- compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
+++ compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
@@ -408,10 +408,12 @@
   }
 
   const char *binary_name = path ? StripModuleName(path) : "";
+  static const char kLLVMSymbolizerPrefix[] = "llvm-symbolizer";
   if (path && path[0] == '\0') {
     VReport(2, "External symbolizer is explicitly disabled.\n");
     return nullptr;
-  } else if (!internal_strcmp(binary_name, "llvm-symbolizer")) {
+  } else if (!internal_strncmp(binary_name, kLLVMSymbolizerPrefix,
+                               internal_strlen(kLLVMSymbolizerPrefix))) {
     VReport(2, "Using llvm-symbolizer at user-specified path: %s\n", path);
     return new(*allocator) LLVMSymbolizer(path, allocator);
   } else if (!internal_strcmp(binary_name, "atos")) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97682.330122.patch
Type: text/x-patch
Size: 981 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210312/0658e4db/attachment.bin>


More information about the llvm-commits mailing list