[compiler-rt] [symbolizer] Don't threat symbolizer API as optional (PR #76103)

via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 13:45:32 PST 2023


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 7ffad37c8694ef2ed554074bc6c622e7ceeb4624 85edda91a2c476fc44b90f6f96decae92102c66a -- compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
index 3b61eb4299..230daedf33 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cpp
@@ -347,7 +347,7 @@ class InternalSymbolizer final : public SymbolizerTool {
       return nullptr;
     CHECK(__sanitizer_symbolize_set_demangle(common_flags()->demangle));
     CHECK(__sanitizer_symbolize_set_inline_frames(
-          common_flags()->symbolize_inline_frames));
+        common_flags()->symbolize_inline_frames));
     return new (*alloc) InternalSymbolizer();
   }
 
@@ -377,9 +377,7 @@ class InternalSymbolizer final : public SymbolizerTool {
     return result;
   }
 
-  void Flush() override {
-    __sanitizer_symbolize_flush();
-  }
+  void Flush() override { __sanitizer_symbolize_flush(); }
 
   const char *Demangle(const char *name) override {
     if (__sanitizer_symbolize_demangle(name, buffer_, sizeof(buffer_))) {

``````````

</details>


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


More information about the llvm-commits mailing list