Skip to content

[Doc][NFC] Fix Sanitizer Ignore list example errors. #143755

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

qinkunbao
Copy link
Member

No description provided.

Created using spr 1.3.6
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Jun 11, 2025
@qinkunbao qinkunbao requested a review from vitalybuka June 11, 2025 17:38
@llvmbot
Copy link
Member

llvmbot commented Jun 11, 2025

@llvm/pr-subscribers-clang

Author: Qinkun Bao (qinkunbao)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/143755.diff

1 Files Affected:

  • (modified) clang/docs/SanitizerSpecialCaseList.rst (+2-2)
diff --git a/clang/docs/SanitizerSpecialCaseList.rst b/clang/docs/SanitizerSpecialCaseList.rst
index 61b6c55d8e6e4..2c50778d0f491 100644
--- a/clang/docs/SanitizerSpecialCaseList.rst
+++ b/clang/docs/SanitizerSpecialCaseList.rst
@@ -109,13 +109,13 @@ precedence. Here are a few examples.
 .. code-block:: bash
 
   $ cat ignorelist1.txt
-  # test.cc will be instrumented.
+  # test.cc will not be instrumented.
   src:*
   src:*/mylib/*=sanitize
   src:*/mylib/test.cc
 
   $ cat ignorelist2.txt
-  # test.cc will not be instrumented.
+  # test.cc will be instrumented.
   src:*
   src:*/mylib/test.cc
   src:*/mylib/*=sanitize

@qinkunbao qinkunbao changed the title [Doc][NFC] Fix Sanitizer Ignore list errors. [Doc][NFC] Fix Sanitizer Ignore list example errors. Jun 11, 2025
@qinkunbao qinkunbao requested a review from thurstond June 12, 2025 14:10
@thurstond
Copy link
Contributor

Just to double-check: this documentation update reflects what actually happens in practice (not just how it's intended to work)?

@qinkunbao
Copy link
Member Author

Just to double-check: this documentation update reflects what actually happens in practice (not just how it's intended to work)?

Yes, after this PR, the document will match the existing implementation.

@qinkunbao qinkunbao merged commit fd88aef into main Jun 12, 2025
10 checks passed
@qinkunbao qinkunbao deleted the users/qinkunbao/spr/docnfc-fix-sanitizer-ignore-list-errors branch June 12, 2025 18:08
tomtor pushed a commit to tomtor/llvm-project that referenced this pull request Jun 14, 2025
akuhlens pushed a commit to akuhlens/llvm-project that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants