[clang] [CIR][NFC] Fix an unused variable warning (PR #137466)
Amr Hesham via cfe-commits
cfe-commits at lists.llvm.org
Sat Apr 26 10:00:52 PDT 2025
https://github.com/AmrDeveloper created https://github.com/llvm/llvm-project/pull/137466
This fixes a warning where a variable assigned in 'if' statement wasn't referenced again.
>From 72690becb4274fb520e0bade5db2a7ef5cd66ab0 Mon Sep 17 00:00:00 2001
From: AmrDeveloper <amr96 at programmer.net>
Date: Sat, 26 Apr 2025 19:00:03 +0200
Subject: [PATCH] [CIR][NFC] Fix an unused variable warning
---
clang/lib/CIR/CodeGen/CIRGenExpr.cpp | 6 +++---
clang/lib/CIR/CodeGen/CIRGenModule.cpp | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/clang/lib/CIR/CodeGen/CIRGenExpr.cpp b/clang/lib/CIR/CodeGen/CIRGenExpr.cpp
index 0a518c0fd935d..ff14798b9d34c 100644
--- a/clang/lib/CIR/CodeGen/CIRGenExpr.cpp
+++ b/clang/lib/CIR/CodeGen/CIRGenExpr.cpp
@@ -311,7 +311,7 @@ LValue CIRGenFunction::emitLValueForField(LValue base, const FieldDecl *field) {
assert(!cir::MissingFeatures::opTBAA());
Address addr = base.getAddress();
- if (auto *classDef = dyn_cast<CXXRecordDecl>(rec)) {
+ if (isa<CXXRecordDecl>(rec)) {
cgm.errorNYI(field->getSourceRange(), "emitLValueForField: C++ class");
return LValue();
}
@@ -701,7 +701,7 @@ CIRGenFunction::emitArraySubscriptExpr(const clang::ArraySubscriptExpr *e) {
}
LValue CIRGenFunction::emitMemberExpr(const MemberExpr *e) {
- if (auto *vd = dyn_cast<VarDecl>(e->getMemberDecl())) {
+ if (isa<VarDecl>(e->getMemberDecl())) {
cgm.errorNYI(e->getSourceRange(), "emitMemberExpr: VarDecl");
return LValue();
}
@@ -734,7 +734,7 @@ LValue CIRGenFunction::emitMemberExpr(const MemberExpr *e) {
return lv;
}
- if (const auto *fd = dyn_cast<FunctionDecl>(nd)) {
+ if (isa<FunctionDecl>(nd)) {
cgm.errorNYI(e->getSourceRange(), "emitMemberExpr: FunctionDecl");
return LValue();
}
diff --git a/clang/lib/CIR/CodeGen/CIRGenModule.cpp b/clang/lib/CIR/CodeGen/CIRGenModule.cpp
index 8aa57d1dbf9b3..b9cc2da583009 100644
--- a/clang/lib/CIR/CodeGen/CIRGenModule.cpp
+++ b/clang/lib/CIR/CodeGen/CIRGenModule.cpp
@@ -366,7 +366,7 @@ void CIRGenModule::emitGlobalVarDefinition(const clang::VarDecl *vd,
bool isTentative) {
const QualType astTy = vd->getType();
const mlir::Type type = convertType(vd->getType());
- if (clang::IdentifierInfo *identifier = vd->getIdentifier()) {
+ if (vd->getIdentifier()) {
StringRef name = getMangledName(GlobalDecl(vd));
auto varOp =
builder.create<cir::GlobalOp>(getLoc(vd->getSourceRange()), name, type);
More information about the cfe-commits
mailing list