https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/149340
getArrayIndex(), getArrayRangeStart(), and getArrayRangeEnd() already
return Expr *.
>From 945f52459c36663ac5c267779ae8e260dd9f9066 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <k...@google.com>
Date: Thu, 17 Jul 2025 08:13:01 -0700
Subject: [PATCH] [Sema] Remove unnecessary casts (NFC)
getArrayIndex(), getArrayRangeStart(), and getArrayRangeEnd() already
return Expr *.
---
clang/lib/Sema/SemaInit.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp
index 95746b35f71ef..1c6f292454ed6 100644
--- a/clang/lib/Sema/SemaInit.cpp
+++ b/clang/lib/Sema/SemaInit.cpp
@@ -3572,7 +3572,7 @@ ExprResult Sema::ActOnDesignatedInitializer(Designation
&Desig,
Designators.push_back(ASTDesignator::CreateFieldDesignator(
D.getFieldDecl(), D.getDotLoc(), D.getFieldLoc()));
} else if (D.isArrayDesignator()) {
- Expr *Index = static_cast<Expr *>(D.getArrayIndex());
+ Expr *Index = D.getArrayIndex();
llvm::APSInt IndexValue;
if (!Index->isTypeDependent() && !Index->isValueDependent())
Index = CheckArrayDesignatorExpr(*this, Index, IndexValue).get();
@@ -3584,8 +3584,8 @@ ExprResult Sema::ActOnDesignatedInitializer(Designation
&Desig,
InitExpressions.push_back(Index);
}
} else if (D.isArrayRangeDesignator()) {
- Expr *StartIndex = static_cast<Expr *>(D.getArrayRangeStart());
- Expr *EndIndex = static_cast<Expr *>(D.getArrayRangeEnd());
+ Expr *StartIndex = D.getArrayRangeStart();
+ Expr *EndIndex = D.getArrayRangeEnd();
llvm::APSInt StartValue;
llvm::APSInt EndValue;
bool StartDependent = StartIndex->isTypeDependent() ||
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
