[PATCH] D24974: Add build script for symbolizer which can be linked into instrumented process.

Evgeniy Stepanov via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 11:38:55 PDT 2016


eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: lib/sanitizer_common/symbolizer/sanitizer_symbolize.cc:21
@@ +20,3 @@
+static llvm::symbolize::LLVMSymbolizer *getDefaultSymbolizer() {
+  static llvm::symbolize::LLVMSymbolizer *DefaultSymbolizer = nullptr;
+  if (!DefaultSymbolizer)
----------------
I don't think you need this initializer at all.


https://reviews.llvm.org/D24974





More information about the llvm-commits mailing list