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

Thurston Dang via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 13:51:05 PST 2023


================
@@ -341,15 +341,14 @@ __sanitizer_symbolize_set_inline_frames(bool InlineFrames);
 class InternalSymbolizer final : public SymbolizerTool {
  public:
   static InternalSymbolizer *get(LowLevelAllocator *alloc) {
-    if (&__sanitizer_symbolize_set_demangle)
-      CHECK(__sanitizer_symbolize_set_demangle(common_flags()->demangle));
-    if (&__sanitizer_symbolize_set_inline_frames)
-      CHECK(__sanitizer_symbolize_set_inline_frames(
+    // These one is the most used one, so we will use it to detect a presense of
----------------
thurstond wrote:

Nit: "presence"

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


More information about the llvm-commits mailing list