[clang] [Sema] Remove unnecessary casts (NFC) (PR #149340)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 17 08:43:13 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
getArrayIndex(), getArrayRangeStart(), and getArrayRangeEnd() already
return Expr *.
---
Full diff: https://github.com/llvm/llvm-project/pull/149340.diff
1 Files Affected:
- (modified) clang/lib/Sema/SemaInit.cpp (+3-3)
``````````diff
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() ||
``````````
</details>
https://github.com/llvm/llvm-project/pull/149340
More information about the cfe-commits
mailing list