[clang] 4784585 - [clang][bytecode][NFC] Remove unnecessary local variable (#152468)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 7 03:39:34 PDT 2025
Author: Timm Baeder
Date: 2025-08-07T12:39:32+02:00
New Revision: 4784585747423a8ed6e3acbe3c8fbe97ba362cc5
URL: https://github.com/llvm/llvm-project/commit/4784585747423a8ed6e3acbe3c8fbe97ba362cc5
DIFF: https://github.com/llvm/llvm-project/commit/4784585747423a8ed6e3acbe3c8fbe97ba362cc5.diff
LOG: [clang][bytecode][NFC] Remove unnecessary local variable (#152468)
Desc is only used once and we can get that information from the Block as
well.
Added:
Modified:
clang/lib/AST/ByteCode/InterpState.cpp
Removed:
################################################################################
diff --git a/clang/lib/AST/ByteCode/InterpState.cpp b/clang/lib/AST/ByteCode/InterpState.cpp
index a06b125b3dace..b3c0a67f8ff3e 100644
--- a/clang/lib/AST/ByteCode/InterpState.cpp
+++ b/clang/lib/AST/ByteCode/InterpState.cpp
@@ -76,9 +76,6 @@ bool InterpState::reportOverflow(const Expr *E, const llvm::APSInt &Value) {
void InterpState::deallocate(Block *B) {
assert(B);
- const Descriptor *Desc = B->getDescriptor();
- assert(Desc);
-
// The block might have a pointer saved in a field in its data
// that points to the block itself. We call the dtor first,
// which will destroy all the data but leave InlineDescriptors
@@ -95,7 +92,7 @@ void InterpState::deallocate(Block *B) {
auto *D = new (Memory) DeadBlock(DeadBlocks, B);
// Since the block doesn't hold any actual data anymore, we can just
// memcpy() everything over.
- std::memcpy(D->rawData(), B->rawData(), Desc->getAllocSize());
+ std::memcpy(D->rawData(), B->rawData(), B->getSize());
D->B.IsInitialized = B->IsInitialized;
// We moved the contents over to the DeadBlock.
More information about the cfe-commits
mailing list