[clang] [Clang] Re-land Overflow Pattern Exclusions (PR #104889)
Vitaly Buka via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 10:29:27 PDT 2024
================
@@ -293,6 +293,48 @@ To silence reports from unsigned integer overflow, you can set
``-fsanitize-recover=unsigned-integer-overflow``, is particularly useful for
providing fuzzing signal without blowing up logs.
+Disabling instrumentation for common overflow patterns
+------------------------------------------------------
+
+There are certain overflow-dependent or overflow-prone code patterns which
+produce a lot of noise for integer overflow/truncation sanitizers. Negated
+unsigned constants, post-decrements in a while loop condition and simple
+overflow checks are accepted and pervasive code patterns. However, the signal
+received from sanitizers instrumenting these code patterns may be too noisy for
+some projects. To disable instrumentation for these common patterns one should
+use ``-fsanitize-undefined-ignore-overflow-pattern=``.
+
+Currently, this option supports three overflow-dependent code idioms:
+
+``negated-unsigned-const``
+
+.. code-block:: c++
+
+ /// -fsanitize-undefined-ignore-overflow-pattern=negated-unsigned-const
+ unsigned long foo = -1UL; // No longer causes a negation overflow warning
+ unsigned long bar = -2UL; // and so on...
+
+``post-decr-while``
+
+.. code-block:: c++
+
+ /// -fsanitize-undefined-ignore-overflow-pattern=post-decr-while
+ unsigned char count = 16;
+ while (count--) { /* ... */ } // No longer causes unsigned-integer-overflow sanitizer to trip
+
+``add-overflow-test``
----------------
vitalybuka wrote:
with `fsanitize-undefined-ignore-overflow-pattern=` name
another `overflow` seems redundant in name
https://github.com/llvm/llvm-project/pull/104889
More information about the cfe-commits
mailing list