================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to