[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