[compiler-rt] [sanitizers] improve debug output for failed suppression parse (PR #72066)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun Nov 12 12:16:46 PST 2023
    
    
  
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: None (schenker)
<details>
<summary>Changes</summary>
If a sanitizer suppression file can not be parsed, add the supported suppression types to the error message.
See https://github.com/llvm/llvm-project/issues/72060.
---
Full diff: https://github.com/llvm/llvm-project/pull/72066.diff
1 Files Affected:
- (modified) compiler-rt/lib/sanitizer_common/sanitizer_suppressions.cpp (+4) 
``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_suppressions.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_suppressions.cpp
index f3818526baab1b7..a94178828bd80db 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_suppressions.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_suppressions.cpp
@@ -139,6 +139,10 @@ void SuppressionContext::Parse(const char *str) {
       }
       if (type == suppression_types_num_) {
         Printf("%s: failed to parse suppressions\n", SanitizerToolName);
+        Printf("Supported suppression types are:\n");
+        for (type = 0; type < suppression_types_num_; type++) {
+          Printf("- %s\n", suppression_types_[type]);
+        }
         Die();
       }
       Suppression s;
``````````
</details>
https://github.com/llvm/llvm-project/pull/72066
    
    
More information about the llvm-commits
mailing list