From 15c60c83aabf515c80d5954c173cc5352c122df6 Mon Sep 17 00:00:00 2001 From: Christian Hecken <104357222+Christian-Hecken@users.noreply.github.com> Date: Tue, 10 Mar 2026 22:57:10 +0100 Subject: [PATCH] Internals: Remove m_mask from VerilatedVpioVar (#7232) Unused since #5573 --- include/verilated_vpi.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/include/verilated_vpi.cpp b/include/verilated_vpi.cpp index ce0efb84e..9c5de2f18 100644 --- a/include/verilated_vpi.cpp +++ b/include/verilated_vpi.cpp @@ -362,10 +362,6 @@ public: class VerilatedVpioVar VL_NOT_FINAL : public VerilatedVpioVarBase { uint8_t* m_prevDatap = nullptr; // Previous value of data, for cbValueChange - union { - uint8_t u8[4]; - uint32_t u32; - } m_mask; // memoized variable mask uint32_t m_entSize = 0; // memoized variable size uint32_t m_bitOffset = 0; int32_t m_partselBits = -1; // Part-select width, -1 means no part-select active @@ -377,21 +373,17 @@ protected: public: VerilatedVpioVar(const VerilatedVar* varp, const VerilatedScope* scopep) : VerilatedVpioVarBase{varp, scopep} { - m_mask.u32 = VL_MASK_I(varp->entBits()); m_entSize = varp->entSize(); m_varDatap = varp->datap(); } explicit VerilatedVpioVar(const VerilatedVpioVar* varp) : VerilatedVpioVarBase{varp} { if (varp) { - m_mask.u32 = varp->m_mask.u32; m_entSize = varp->m_entSize; m_varDatap = varp->m_varDatap; m_index = varp->m_index; m_partselBits = varp->m_partselBits; // Not copying m_prevDatap, must be nullptr - } else { - m_mask.u32 = 0; } } ~VerilatedVpioVar() override { @@ -410,8 +402,6 @@ public: if (m_partselBits >= 0) return static_cast(m_partselBits); return VerilatedVpioVarBase::size(); } - uint32_t mask() const { return m_mask.u32; } - uint8_t mask_byte(int idx) const { return m_mask.u8[idx & 3]; } uint32_t entSize() const { return m_entSize; } const std::vector& index() const { return m_index; } // Create a part-selected view of this variable with the given bit range [hi:lo].