[all-commits] [llvm/llvm-project] f8ae2e: Reland: [sanitizer_symbolizer] Add StackTracePrint...

Andres Villegas via All-commits all-commits at lists.llvm.org
Thu Sep 21 11:28:33 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f8ae2e4277acae7d2e147b9e86ad8bafc1ef10e9
      https://github.com/llvm/llvm-project/commit/f8ae2e4277acae7d2e147b9e86ad8bafc1ef10e9
  Author: Andres Villegas <andresvi at google.com>
  Date:   2023-09-21 (Thu, 21 Sep 2023)

  Changed paths:
    M compiler-rt/lib/hwasan/hwasan_report.cpp
    M compiler-rt/lib/msan/msan_report.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_stacktrace_libcdep.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_stacktrace_printer.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_stacktrace_printer.h
    M compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_markup.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_report.cpp
    M compiler-rt/lib/sanitizer_common/tests/sanitizer_stacktrace_printer_test.cpp
    M compiler-rt/lib/tsan/rtl/tsan_report.cpp
    M compiler-rt/lib/ubsan/ubsan_diag.cpp

  Log Message:
  -----------
  Reland: [sanitizer_symbolizer] Add StackTracePrinter virtual class (#66689)

Introduce a new virtual class StackTracePrinter and an implementation
FormattedStackTracePrinter in preparation of enabling symbolizer markup
for linux.
This change allows us to implement other behaviour under the same api
for StackTracePrinter, for example, MarkupStackTracePrinter.

Reason for revert: A missing header file for the
sanitizer_symbolizer_markup.cpp files.
This was not caught in local builds or pre-merge checks given that to
trigger the error, the code
has to be compiled for Fuchsia.
For this reland I've build for the fuchsia targets as well as linux.




More information about the All-commits mailing list