================ @@ -80,16 +80,19 @@ UseIntegerSignComparisonCheck::UseIntegerSignComparisonCheck( : ClangTidyCheck(Name, Context), IncludeInserter(Options.getLocalOrGlobal("IncludeStyle", utils::IncludeSorter::IS_LLVM), - areDiagsSelfContained()) {} + areDiagsSelfContained()), + ConsideringIntSize(Options.get("ConsideringIntegerSize", true)) {} ---------------- PiotrZSL wrote:
use more straightforward option name, like "CheckIntegerSize" or something similar. I just dont like how this "Considering" sounds in context of an option, even "Consider" would sound better. https://github.com/llvm/llvm-project/pull/121506 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits