llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> We often don't need it (especially in C), so make this optional and create it only when we first allocate something. --- Full diff: https://github.com/llvm/llvm-project/pull/158802.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/InterpState.h (+4-2) ``````````diff diff --git a/clang/lib/AST/ByteCode/InterpState.h b/clang/lib/AST/ByteCode/InterpState.h index e095908bce986..4e1b9aec09099 100644 --- a/clang/lib/AST/ByteCode/InterpState.h +++ b/clang/lib/AST/ByteCode/InterpState.h @@ -122,7 +122,9 @@ class InterpState final : public State, public SourceMapper { StdAllocatorCaller getStdAllocatorCaller(StringRef Name) const; void *allocate(size_t Size, unsigned Align = 8) const { - return Allocator.Allocate(Size, Align); + if (!Allocator) + Allocator.emplace(); + return Allocator->Allocate(Size, Align); } template <typename T> T *allocate(size_t Num = 1) const { return static_cast<T *>(allocate(Num * sizeof(T), alignof(T))); @@ -188,7 +190,7 @@ class InterpState final : public State, public SourceMapper { /// for. llvm::SmallVector<const Block *> InitializingBlocks; - mutable llvm::BumpPtrAllocator Allocator; + mutable std::optional<llvm::BumpPtrAllocator> Allocator; }; class InterpStateCCOverride final { `````````` </details> https://github.com/llvm/llvm-project/pull/158802 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
