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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 11 14:27:10 PDT 2017


This revision was automatically updated to reflect the committed changes.
vitalybuka marked an inline comment as done.
Closed by commit rL312954: [ubsan] Make ubsan version of __sanitizer_print_stack_trace consistent with… (authored by vitalybuka).

Changed prior to commit:
  https://reviews.llvm.org/D37657?vs=114695&id=114700#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D37657

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


Index: compiler-rt/trunk/test/sanitizer_common/TestCases/print-stack-trace.cc
===================================================================
--- compiler-rt/trunk/test/sanitizer_common/TestCases/print-stack-trace.cc
+++ compiler-rt/trunk/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/trunk/lib/ubsan/ubsan_diag_standalone.cc
===================================================================
--- compiler-rt/trunk/lib/ubsan/ubsan_diag_standalone.cc
+++ compiler-rt/trunk/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.114700.patch
Type: text/x-patch
Size: 1187 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170911/33973107/attachment.bin>


More information about the llvm-commits mailing list