Skip to content

Commit 9db6c32

Browse files
authored
[CIR][NFC] Fix an unused variable warning (#140783)
This fixes a warning where a variable assigned in 'if' statement wasn't referenced again, and where else is used when 'if' has returns statement in the if-else statement
1 parent 9459c83 commit 9db6c32

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

clang/lib/CIR/CodeGen/CIRGenCXXExpr.cpp

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ RValue CIRGenFunction::emitCXXMemberOrOperatorMemberCallExpr(
115115
thisPtr = emitLValue(base);
116116
}
117117

118-
if (const CXXConstructorDecl *ctor = dyn_cast<CXXConstructorDecl>(md)) {
118+
if (isa<CXXConstructorDecl>(md)) {
119119
cgm.errorNYI(ce->getSourceRange(),
120120
"emitCXXMemberOrOperatorMemberCallExpr: constructor call");
121121
return RValue::get(nullptr);
@@ -129,29 +129,29 @@ RValue CIRGenFunction::emitCXXMemberOrOperatorMemberCallExpr(
129129
cgm.errorNYI(ce->getSourceRange(),
130130
"emitCXXMemberOrOperatorMemberCallExpr: trivial assignment");
131131
return RValue::get(nullptr);
132-
} else {
133-
assert(md->getParent()->mayInsertExtraPadding() &&
134-
"unknown trivial member function");
135132
}
133+
134+
assert(md->getParent()->mayInsertExtraPadding() &&
135+
"unknown trivial member function");
136136
}
137137

138138
// Compute the function type we're calling
139139
const CXXMethodDecl *calleeDecl = md;
140140
const CIRGenFunctionInfo *fInfo = nullptr;
141-
if (const auto *dtor = dyn_cast<CXXDestructorDecl>(calleeDecl)) {
141+
if (isa<CXXDestructorDecl>(calleeDecl)) {
142142
cgm.errorNYI(ce->getSourceRange(),
143143
"emitCXXMemberOrOperatorMemberCallExpr: destructor call");
144144
return RValue::get(nullptr);
145-
} else {
146-
fInfo = &cgm.getTypes().arrangeCXXMethodDeclaration(calleeDecl);
147145
}
148146

147+
fInfo = &cgm.getTypes().arrangeCXXMethodDeclaration(calleeDecl);
148+
149149
mlir::Type ty = cgm.getTypes().getFunctionType(*fInfo);
150150

151151
assert(!cir::MissingFeatures::sanitizers());
152152
assert(!cir::MissingFeatures::emitTypeCheck());
153153

154-
if (const auto *dtor = dyn_cast<CXXDestructorDecl>(calleeDecl)) {
154+
if (isa<CXXDestructorDecl>(calleeDecl)) {
155155
cgm.errorNYI(ce->getSourceRange(),
156156
"emitCXXMemberOrOperatorMemberCallExpr: destructor call");
157157
return RValue::get(nullptr);

0 commit comments

Comments
 (0)