[compiler-rt] [sanitizer_symbolizer] Add initial symbolizer markup (PR #72605)

via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 17 13:22:48 PST 2023


=?utf-8?q?Andrés?= Villegas <andresvi at google.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/72605 at github.com>


================
@@ -322,11 +327,12 @@ void FormattedStackTracePrinter::RenderData(InternalScopedString *buffer,
   }
 }
 
-#endif  // !SANITIZER_SYMBOLIZER_MARKUP
+#endif  // !SANITIZER_SYMBOLIZER_MARKUP_FUCHSIA
 
-void FormattedStackTracePrinter::RenderSourceLocation(
-    InternalScopedString *buffer, const char *file, int line, int column,
-    bool vs_style, const char *strip_path_prefix) {
+void StackTracePrinter::RenderSourceLocation(InternalScopedString *buffer,
+                                             const char *file, int line,
+                                             int column, bool vs_style,
+                                             const char *strip_path_prefix) {
----------------
PiJoules wrote:

Since this and `RenderModuleLocation` don't seem to be changed, it might be simpler/easier to read this patch if these changes were in a separate NFC patch.

https://github.com/llvm/llvm-project/pull/72605


More information about the llvm-commits mailing list