[clang] [clang][NFC] Clean up Expr::isTemporaryObject() (PR #140229)
via cfe-commits
cfe-commits at lists.llvm.org
Fri May 16 02:45:01 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/140229.diff
1 Files Affected:
- (modified) clang/lib/AST/Expr.cpp (+3-3)
``````````diff
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index 8557c3b82ca39..0ad73360dc0ff 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -3263,8 +3263,8 @@ bool Expr::isTemporaryObject(ASTContext &C, const CXXRecordDecl *TempTy) const {
// refer to temporaries of that type:
// - implicit derived-to-base conversions
- if (isa<ImplicitCastExpr>(E)) {
- switch (cast<ImplicitCastExpr>(E)->getCastKind()) {
+ if (const auto *ICE = dyn_cast<ImplicitCastExpr>(E))
+ switch (ICE->getCastKind()) {
case CK_DerivedToBase:
case CK_UncheckedDerivedToBase:
return false;
@@ -3277,7 +3277,7 @@ bool Expr::isTemporaryObject(ASTContext &C, const CXXRecordDecl *TempTy) const {
if (isa<MemberExpr>(E))
return false;
- if (const BinaryOperator *BO = dyn_cast<BinaryOperator>(E))
+ if (const auto *BO = dyn_cast<BinaryOperator>(E))
if (BO->isPtrMemOp())
return false;
``````````
</details>
https://github.com/llvm/llvm-project/pull/140229
More information about the cfe-commits
mailing list