[PATCH] [Sanitizer] Introduce generic stack frame rendering machinery

Alexey Samsonov vonosmas at gmail.com
Wed Nov 5 14:35:49 PST 2014


================
Comment at: lib/sanitizer_common/sanitizer_stacktrace_printer.cc:26
@@ +25,3 @@
+
+const char kDefaultFormat[] = "    #%n %p %F %L";
+
----------------
kcc wrote:
> static
Done

================
Comment at: lib/sanitizer_common/sanitizer_stacktrace_printer.cc:31
@@ +30,3 @@
+                 const char *strip_func_prefix) {
+  if (0 == internal_strcmp(format, "DEFAULT"))
+    format = kDefaultFormat;
----------------
kcc wrote:
> why not nullptr? 
Do you mean nullptr value for "format" string? Well, in future it will be a runtime flag, specified by the user, so I thought it would be nice to make this special case have more meaningful representation.

================
Comment at: lib/sanitizer_common/sanitizer_stacktrace_printer.h:10
@@ +9,3 @@
+//
+// This file is shared between AddressSanitizer and ThreadSanitizer
+// run-time libraries.
----------------
kcc wrote:
> ..., and other sanitizers
Done

http://reviews.llvm.org/D6140






More information about the llvm-commits mailing list