[clang-tools-extra] [clang-tidy] Add QtEnabled option to modernize-use-integer-sign-comprison (PR #122127)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 11 00:38:44 PST 2025


================
@@ -154,8 +156,16 @@ void UseIntegerSignComparisonCheck::check(
   DiagnosticBuilder Diag =
       diag(BinaryOp->getBeginLoc(),
            "comparison between 'signed' and 'unsigned' integers");
-  const std::string CmpNamespace = ("std::" + parseOpCode(OpCode)).str();
-  const std::string CmpHeader = "<utility>";
+  std::string CmpNamespace;
+  std::string CmpHeader;
----------------
PiotrZSL wrote:

Tip: change CmpHeader  from std::string to llvm::StringRef

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


More information about the cfe-commits mailing list