[clang] [clang][analyzer] Add checker 'alpha.core.FixedAddressDereference' (PR #127191)

DonĂ¡t Nagy via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 21 08:28:06 PST 2025


================
@@ -395,6 +395,19 @@ ANALYZER_OPTION(
     "flex\" won't be analyzed.",
     true)
 
+ANALYZER_OPTION(
+    bool, ShouldSuppressAddressSpaces, "suppress-all-address-spaces",
----------------
NagyDonat wrote:

The name of this analyzer option is misleading: the natural guess is that it makes the analyzer ignore the address space annotations -- and it's completely counter-intuitive that it suppresses certain results that come from two particular checkers. Of course the attached documentation describes its meaning, but I don't think that's enough (especially considering that these option docs are currently very obscure -- e.g. they don't appear on the webpage.

Note that this name was already misleading when it belonged to just a checker option, but moving it into the global namespace exacerbates the problem.

I know that it's difficult to find a more accurate name, but perhaps something like `ignore-deref-from-all-address-spaces` could work.

@steakhal What do you think about this?

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


More information about the cfe-commits mailing list