<div dir="ltr">Bill, please also merge this into the 3.4 branch.<div><br></div><div>-Chandler</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Dec 9, 2013 at 8:40 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: chandlerc<br>
Date: Mon Dec 9 22:40:39 2013<br>
New Revision: 196875<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=196875&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=196875&view=rev</a><br>
Log:<br>
Revert a final patch that was committed without the author contributing<br>
it to the LLVM project through the appropriate channels.<br>
<br>
This reverts:<br>
r195837: "[Sanitizer] Add rudimentary support for using libbacktrace in ..."<br>
<br>
Removed:<br>
compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc<br>
compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h<br>
Modified:<br>
compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt<br>
compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc<br>
<br>
Modified: compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt?rev=196875&r1=196874&r2=196875&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt?rev=196875&r1=196874&r2=196875&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt (original)<br>
+++ compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt Mon Dec 9 22:40:39 2013<br>
@@ -28,7 +28,6 @@ set(SANITIZER_LIBCDEP_SOURCES<br>
sanitizer_posix_libcdep.cc<br>
sanitizer_stacktrace_libcdep.cc<br>
sanitizer_stoptheworld_linux_libcdep.cc<br>
- sanitizer_symbolizer_libbacktrace.cc<br>
sanitizer_symbolizer_libcdep.cc<br>
sanitizer_symbolizer_posix_libcdep.cc)<br>
<br>
@@ -66,7 +65,6 @@ set(SANITIZER_HEADERS<br>
sanitizer_stoptheworld.h<br>
sanitizer_suppressions.h<br>
sanitizer_symbolizer.h<br>
- sanitizer_symbolizer_libbacktrace.h<br>
sanitizer_syscall_generic.inc<br>
sanitizer_syscall_linux_x86_64.inc<br>
sanitizer_thread_registry.h)<br>
<br>
Removed: compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc?rev=196874&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc?rev=196874&view=auto</a><br>
==============================================================================<br>
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc (original)<br>
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc (removed)<br>
@@ -1,146 +0,0 @@<br>
-//===-- sanitizer_symbolizer_libbacktrace.cc ------------------------------===//<br>
-//<br>
-// The LLVM Compiler Infrastructure<br>
-//<br>
-// This file is distributed under the University of Illinois Open Source<br>
-// License. See LICENSE.TXT for details.<br>
-//<br>
-//===----------------------------------------------------------------------===//<br>
-//<br>
-// This file is shared between AddressSanitizer and ThreadSanitizer<br>
-// run-time libraries.<br>
-// Libbacktrace implementation of symbolizer parts.<br>
-//===----------------------------------------------------------------------===//<br>
-<br>
-#include "sanitizer_platform.h"<br>
-<br>
-#include "sanitizer_internal_defs.h"<br>
-#include "sanitizer_symbolizer.h"<br>
-#include "sanitizer_symbolizer_libbacktrace.h"<br>
-<br>
-#if SANITIZER_LIBBACKTRACE<br>
-# include "backtrace-supported.h"<br>
-# if SANITIZER_POSIX && BACKTRACE_SUPPORTED && !BACKTRACE_USES_MALLOC<br>
-# include "backtrace.h"<br>
-# else<br>
-# define SANITIZER_LIBBACKTRACE 0<br>
-# endif<br>
-#endif<br>
-<br>
-namespace __sanitizer {<br>
-<br>
-#if SANITIZER_LIBBACKTRACE<br>
-<br>
-namespace {<br>
-<br>
-struct SymbolizeCodeData {<br>
- AddressInfo *frames;<br>
- uptr n_frames;<br>
- uptr max_frames;<br>
- const char *module_name;<br>
- uptr module_offset;<br>
-};<br>
-<br>
-extern "C" {<br>
-static int SymbolizeCodePCInfoCallback(void *vdata, uintptr_t addr,<br>
- const char *filename, int lineno,<br>
- const char *function) {<br>
- SymbolizeCodeData *cdata = (SymbolizeCodeData *)vdata;<br>
- if (function) {<br>
- AddressInfo *info = &cdata->frames[cdata->n_frames++];<br>
- info->Clear();<br>
- info->FillAddressAndModuleInfo(addr, cdata->module_name,<br>
- cdata->module_offset);<br>
- info->function = internal_strdup(function);<br>
- if (filename)<br>
- info->file = internal_strdup(filename);<br>
- info->line = lineno;<br>
- if (cdata->n_frames == cdata->max_frames)<br>
- return 1;<br>
- }<br>
- return 0;<br>
-}<br>
-<br>
-static void SymbolizeCodeCallback(void *vdata, uintptr_t addr,<br>
- const char *symname, uintptr_t, uintptr_t) {<br>
- SymbolizeCodeData *cdata = (SymbolizeCodeData *)vdata;<br>
- if (symname) {<br>
- AddressInfo *info = &cdata->frames[0];<br>
- info->Clear();<br>
- info->FillAddressAndModuleInfo(addr, cdata->module_name,<br>
- cdata->module_offset);<br>
- info->function = internal_strdup(symname);<br>
- cdata->n_frames = 1;<br>
- }<br>
-}<br>
-<br>
-static void SymbolizeDataCallback(void *vdata, uintptr_t, const char *symname,<br>
- uintptr_t symval, uintptr_t symsize) {<br>
- DataInfo *info = (DataInfo *)vdata;<br>
- if (symname && symval) {<br>
- info->name = internal_strdup(symname);<br>
- info->start = symval;<br>
- info->size = symsize;<br>
- }<br>
-}<br>
-<br>
-static void ErrorCallback(void *, const char *, int) {}<br>
-} // extern "C"<br>
-<br>
-} // namespace<br>
-<br>
-LibbacktraceSymbolizer *LibbacktraceSymbolizer::get(LowLevelAllocator *alloc) {<br>
- // State created in backtrace_create_state is leaked.<br>
- void *state = (void *)(backtrace_create_state("/proc/self/exe", 0,<br>
- ErrorCallback, NULL));<br>
- if (!state)<br>
- return 0;<br>
- return new(*alloc) LibbacktraceSymbolizer(state);<br>
-}<br>
-<br>
-uptr LibbacktraceSymbolizer::SymbolizeCode(uptr addr, AddressInfo *frames,<br>
- uptr max_frames,<br>
- const char *module_name,<br>
- uptr module_offset) {<br>
- SymbolizeCodeData data;<br>
- data.frames = frames;<br>
- data.n_frames = 0;<br>
- data.max_frames = max_frames;<br>
- data.module_name = module_name;<br>
- data.module_offset = module_offset;<br>
- backtrace_pcinfo((backtrace_state *)state_, addr, SymbolizeCodePCInfoCallback,<br>
- ErrorCallback, &data);<br>
- if (data.n_frames)<br>
- return data.n_frames;<br>
- backtrace_syminfo((backtrace_state *)state_, addr, SymbolizeCodeCallback,<br>
- ErrorCallback, &data);<br>
- return data.n_frames;<br>
-}<br>
-<br>
-bool LibbacktraceSymbolizer::SymbolizeData(DataInfo *info) {<br>
- backtrace_syminfo((backtrace_state *)state_, info->address,<br>
- SymbolizeDataCallback, ErrorCallback, info);<br>
- return true;<br>
-}<br>
-<br>
-#else // SANITIZER_LIBBACKTRACE<br>
-<br>
-LibbacktraceSymbolizer *LibbacktraceSymbolizer::get(LowLevelAllocator *alloc) {<br>
- return 0;<br>
-}<br>
-<br>
-uptr LibbacktraceSymbolizer::SymbolizeCode(uptr addr, AddressInfo *frames,<br>
- uptr max_frames,<br>
- const char *module_name,<br>
- uptr module_offset) {<br>
- (void)state_;<br>
- return 0;<br>
-}<br>
-<br>
-bool LibbacktraceSymbolizer::SymbolizeData(DataInfo *info) {<br>
- return false;<br>
-}<br>
-<br>
-#endif // SANITIZER_LIBBACKTRACE<br>
-<br>
-} // namespace __sanitizer<br>
<br>
Removed: compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h?rev=196874&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h?rev=196874&view=auto</a><br>
==============================================================================<br>
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h (original)<br>
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.h (removed)<br>
@@ -1,40 +0,0 @@<br>
-//===-- sanitizer_symbolizer_libbacktrace.h -------------------------------===//<br>
-//<br>
-// The LLVM Compiler Infrastructure<br>
-//<br>
-// This file is distributed under the University of Illinois Open Source<br>
-// License. See LICENSE.TXT for details.<br>
-//<br>
-//===----------------------------------------------------------------------===//<br>
-//<br>
-// This file is shared between AddressSanitizer and ThreadSanitizer<br>
-// run-time libraries.<br>
-// Header for libbacktrace symbolizer.<br>
-//===----------------------------------------------------------------------===//<br>
-<br>
-#include "sanitizer_platform.h"<br>
-#include "sanitizer_common.h"<br>
-#include "sanitizer_symbolizer.h"<br>
-<br>
-#ifndef SANITIZER_LIBBACKTRACE<br>
-# define SANITIZER_LIBBACKTRACE 0<br>
-#endif<br>
-<br>
-namespace __sanitizer {<br>
-<br>
-class LibbacktraceSymbolizer {<br>
- public:<br>
- static LibbacktraceSymbolizer *get(LowLevelAllocator *alloc);<br>
-<br>
- uptr SymbolizeCode(uptr addr, AddressInfo *frames, uptr max_frames,<br>
- const char *module_name, uptr module_offset);<br>
-<br>
- bool SymbolizeData(DataInfo *info);<br>
-<br>
- private:<br>
- explicit LibbacktraceSymbolizer(void *state) : state_(state) {}<br>
-<br>
- void *state_; // Leaked.<br>
-};<br>
-<br>
-} // namespace __sanitizer<br>
<br>
Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc?rev=196875&r1=196874&r2=196875&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc?rev=196875&r1=196874&r2=196875&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc (original)<br>
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc Mon Dec 9 22:40:39 2013<br>
@@ -21,7 +21,6 @@<br>
#include "sanitizer_placement_new.h"<br>
#include "sanitizer_procmaps.h"<br>
#include "sanitizer_symbolizer.h"<br>
-#include "sanitizer_symbolizer_libbacktrace.h"<br>
<br>
#include <errno.h><br>
#include <stdlib.h><br>
@@ -387,12 +386,10 @@ class InternalSymbolizer {<br>
class POSIXSymbolizer : public Symbolizer {<br>
public:<br>
POSIXSymbolizer(ExternalSymbolizer *external_symbolizer,<br>
- InternalSymbolizer *internal_symbolizer,<br>
- LibbacktraceSymbolizer *libbacktrace_symbolizer)<br>
+ InternalSymbolizer *internal_symbolizer)<br>
: Symbolizer(),<br>
external_symbolizer_(external_symbolizer),<br>
- internal_symbolizer_(internal_symbolizer),<br>
- libbacktrace_symbolizer_(libbacktrace_symbolizer) {}<br>
+ internal_symbolizer_(internal_symbolizer) {}<br>
<br>
uptr SymbolizeCode(uptr addr, AddressInfo *frames, uptr max_frames) {<br>
BlockingMutexLock l(&mu_);<br>
@@ -403,17 +400,9 @@ class POSIXSymbolizer : public Symbolize<br>
return 0;<br>
const char *module_name = module->full_name();<br>
uptr module_offset = addr - module->base_address();<br>
- // First, try to use libbacktrace symbolizer (if it's available).<br>
- if (libbacktrace_symbolizer_ != 0) {<br>
- mu_.CheckLocked();<br>
- uptr res = libbacktrace_symbolizer_->SymbolizeCode(<br>
- addr, frames, max_frames, module_name, module_offset);<br>
- if (res > 0)<br>
- return res;<br>
- }<br>
const char *str = SendCommand(false, module_name, module_offset);<br>
if (str == 0) {<br>
- // Symbolizer was not initialized or failed. Fill only data<br>
+ // External symbolizer was not initialized or failed. Fill only data<br>
// about module name and offset.<br>
AddressInfo *info = &frames[0];<br>
info->Clear();<br>
@@ -474,11 +463,6 @@ class POSIXSymbolizer : public Symbolize<br>
info->address = addr;<br>
info->module = internal_strdup(module_name);<br>
info->module_offset = module_offset;<br>
- if (libbacktrace_symbolizer_ != 0) {<br>
- mu_.CheckLocked();<br>
- if (libbacktrace_symbolizer_->SymbolizeData(info))<br>
- return true;<br>
- }<br>
const char *str = SendCommand(true, module_name, module_offset);<br>
if (str == 0)<br>
return true;<br>
@@ -490,8 +474,7 @@ class POSIXSymbolizer : public Symbolize<br>
}<br>
<br>
bool IsAvailable() {<br>
- return internal_symbolizer_ != 0 || external_symbolizer_ != 0 ||<br>
- libbacktrace_symbolizer_ != 0;<br>
+ return internal_symbolizer_ != 0 || external_symbolizer_ != 0;<br>
}<br>
<br>
bool IsExternalAvailable() {<br>
@@ -584,30 +567,24 @@ class POSIXSymbolizer : public Symbolize<br>
<br>
ExternalSymbolizer *external_symbolizer_; // Leaked.<br>
InternalSymbolizer *const internal_symbolizer_; // Leaked.<br>
- LibbacktraceSymbolizer *libbacktrace_symbolizer_; // Leaked.<br>
};<br>
<br>
Symbolizer *Symbolizer::PlatformInit(const char *path_to_external) {<br>
InternalSymbolizer* internal_symbolizer =<br>
InternalSymbolizer::get(&symbolizer_allocator_);<br>
ExternalSymbolizer *external_symbolizer = 0;<br>
- LibbacktraceSymbolizer *libbacktrace_symbolizer = 0;<br>
<br>
if (!internal_symbolizer) {<br>
- libbacktrace_symbolizer =<br>
- LibbacktraceSymbolizer::get(&symbolizer_allocator_);<br>
- if (!libbacktrace_symbolizer) {<br>
- // Find path to llvm-symbolizer if it's not provided.<br>
- if (!path_to_external)<br>
- path_to_external = FindPathToBinary("llvm-symbolizer");<br>
- if (path_to_external && path_to_external[0] != '\0')<br>
- external_symbolizer = new(symbolizer_allocator_)<br>
- ExternalSymbolizer(path_to_external);<br>
- }<br>
+ // Find path to llvm-symbolizer if it's not provided.<br>
+ if (!path_to_external)<br>
+ path_to_external = FindPathToBinary("llvm-symbolizer");<br>
+ if (path_to_external && path_to_external[0] != '\0')<br>
+ external_symbolizer = new(symbolizer_allocator_)<br>
+ ExternalSymbolizer(path_to_external);<br>
}<br>
<br>
- return new(symbolizer_allocator_) POSIXSymbolizer(<br>
- external_symbolizer, internal_symbolizer, libbacktrace_symbolizer);<br>
+ return new(symbolizer_allocator_)<br>
+ POSIXSymbolizer(external_symbolizer, internal_symbolizer);<br>
}<br>
<br>
} // namespace __sanitizer<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>