[PATCH] D37657: [ubsan] Make ubsan version of __sanitizer_print_stack_trace consistent with other sanitizers

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka created this revision.
Herald added a subscriber: kubamracek.

Other sanitizers include __sanitizer_print_stack_trace into stack trace.


https://reviews.llvm.org/D37657

Files:
  compiler-rt/lib/ubsan/ubsan_diag_standalone.cc
  compiler-rt/test/sanitizer_common/TestCases/print-stack-trace.cc


Index: compiler-rt/test/sanitizer_common/TestCases/print-stack-trace.cc
===================================================================
--- compiler-rt/test/sanitizer_common/TestCases/print-stack-trace.cc
+++ compiler-rt/test/sanitizer_common/TestCases/print-stack-trace.cc
@@ -4,7 +4,6 @@
 // RUN: %env_tool_opts=symbolize_inline_frames=false:stack_trace_format=DEFAULT %run %t 2>&1 | FileCheck %s --check-prefix=NOINLINE
 
 // UNSUPPORTED: darwin
-// XFAIL: ubsan
 
 #include <sanitizer/common_interface_defs.h>
 
Index: compiler-rt/lib/ubsan/ubsan_diag_standalone.cc
===================================================================
--- compiler-rt/lib/ubsan/ubsan_diag_standalone.cc
+++ compiler-rt/lib/ubsan/ubsan_diag_standalone.cc
@@ -26,9 +26,10 @@
   if (request_fast_unwind)
     __sanitizer::GetThreadStackTopAndBottom(false, &top, &bottom);
 
-  GET_REPORT_OPTIONS(false);
+  GET_CURRENT_PC_BP_SP;
+  (void)sp;
   BufferedStackTrace stack;
-  stack.Unwind(kStackTraceMax, Opts.pc, Opts.bp, nullptr, top, bottom,
+  stack.Unwind(kStackTraceMax, pc, bp, nullptr, top, bottom,
                request_fast_unwind);
   stack.Print();
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37657.114475.patch
Type: text/x-patch
Size: 1151 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170909/a8b87d8c/attachment.bin>