llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> Unused these days. --- Full diff: https://github.com/llvm/llvm-project/pull/158656.diff 2 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+5-5) - (modified) clang/lib/AST/ByteCode/Compiler.h (-8) ``````````diff 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) { `````````` </details> https://github.com/llvm/llvm-project/pull/158656 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
