[clang-tools-extra] [clang-tidy][NFC] Clean up and slightly optimize `modernize-use-integer-sign-comparison` (PR #163492)

Victor Chernyakin via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 14 21:26:46 PDT 2025


================
@@ -119,23 +118,16 @@ void UseIntegerSignComparisonCheck::check(
   Expr::EvalResult EVResult;
   if (!SignedCastExpression->isValueDependent() &&
       SignedCastExpression->getSubExpr()->EvaluateAsInt(EVResult,
-                                                        *Result.Context)) {
-    const llvm::APSInt SValue = EVResult.Val.getInt();
----------------
localspook wrote:

`getInt()` returns a reference, so this is an unnecessary copy

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


More information about the cfe-commits mailing list