https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/158656
Unused these days.
>From 5d069297139dcc7bfef03dedb1e7e46c34bf1727 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?= <tbae...@redhat.com>
Date: Mon, 15 Sep 2025 17:00:28 +0200
Subject: [PATCH] [clang][bytecode][NFC] Remove BlockScope
Unused these days.
---
clang/lib/AST/ByteCode/Compiler.cpp | 10 +++++-----
clang/lib/AST/ByteCode/Compiler.h | 8 --------
2 files changed, 5 insertions(+), 13 deletions(-)
diff --git a/clang/lib/AST/ByteCode/Compiler.cpp
b/clang/lib/AST/ByteCode/Compiler.cpp
index 78b74acc3789d..cecdb7b2dda55 100644
--- a/clang/lib/AST/ByteCode/Compiler.cpp
+++ b/clang/lib/AST/ByteCode/Compiler.cpp
@@ -2429,7 +2429,7 @@ bool Compiler<Emitter>::VisitArrayInitLoopExpr(const
ArrayInitLoopExpr *E) {
// and the RHS is our SubExpr.
for (size_t I = 0; I != Size; ++I) {
ArrayIndexScope<Emitter> IndexScope(this, I);
- BlockScope<Emitter> BS(this);
+ LocalScope<Emitter> BS(this);
if (!this->visitArrayElemInit(I, SubExpr, SubExprT))
return false;
@@ -4140,7 +4140,7 @@ bool Compiler<Emitter>::VisitCXXStdInitializerListExpr(
template <class Emitter>
bool Compiler<Emitter>::VisitStmtExpr(const StmtExpr *E) {
- BlockScope<Emitter> BS(this);
+ LocalScope<Emitter> BS(this);
StmtExprScope<Emitter> SS(this);
const CompoundStmt *CS = E->getSubStmt();
@@ -5111,7 +5111,7 @@ bool Compiler<Emitter>::VisitCallExpr(const CallExpr *E) {
}
}
- BlockScope<Emitter> CallScope(this, ScopeKind::Call);
+ LocalScope<Emitter> CallScope(this, ScopeKind::Call);
QualType ReturnType = E->getCallReturnType(Ctx.getASTContext());
OptPrimType T = classify(ReturnType);
@@ -5475,7 +5475,7 @@ template <class Emitter> bool
Compiler<Emitter>::visitStmt(const Stmt *S) {
template <class Emitter>
bool Compiler<Emitter>::visitCompoundStmt(const CompoundStmt *S) {
- BlockScope<Emitter> Scope(this);
+ LocalScope<Emitter> Scope(this);
for (const auto *InnerStmt : S->body())
if (!visitStmt(InnerStmt))
return false;
@@ -6211,7 +6211,7 @@ bool Compiler<Emitter>::compileConstructor(const
CXXConstructorDecl *Ctor) {
InitLinkScope<Emitter> InitScope(this, InitLink::This());
for (const auto *Init : Ctor->inits()) {
// Scope needed for the initializers.
- BlockScope<Emitter> Scope(this);
+ LocalScope<Emitter> Scope(this);
const Expr *InitExpr = Init->getInit();
if (const FieldDecl *Member = Init->getMember()) {
diff --git a/clang/lib/AST/ByteCode/Compiler.h
b/clang/lib/AST/ByteCode/Compiler.h
index 5f392964c076a..09599b3547888 100644
--- a/clang/lib/AST/ByteCode/Compiler.h
+++ b/clang/lib/AST/ByteCode/Compiler.h
@@ -624,14 +624,6 @@ template <class Emitter> class LocalScope : public
VariableScope<Emitter> {
UnsignedOrNone Idx = std::nullopt;
};
-/// Scope for storage declared in a compound statement.
-// FIXME: Remove?
-template <class Emitter> class BlockScope final : public LocalScope<Emitter> {
-public:
- BlockScope(Compiler<Emitter> *Ctx, ScopeKind Kind = ScopeKind::Block)
- : LocalScope<Emitter>(Ctx, Kind) {}
-};
-
template <class Emitter> class ArrayIndexScope final {
public:
ArrayIndexScope(Compiler<Emitter> *Ctx, uint64_t Index) : Ctx(Ctx) {
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
