luke/verilator
1
0
Fork 0
mirror of https://github.com/verilator/verilator.git synced 2026-10-06 10:03:44 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix array slice crash when used as a bare value (#5132) (#8262)

Browse Source
This commit is contained in:
Marco Frank
2026-09-06 20:57:44 -04:00
committed by GitHub
parent 9da900a5e2
commit 63bcfff80a
7 changed files with 125 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/V3Slice.cpp
+17
View File
@@ -382,6 +382,23 @@ class SliceVisitor final : public VNVisitor {
void visit(AstEqCase* nodep) override { expandBiOp(nodep); }
void visit(AstNeqCase* nodep) override { expandBiOp(nodep); }
void visit(AstSliceSel* nodep) override {
// Slice used as a bare value, e.g. a $display argument. Build it via
// VlUnpacked::slice<N_Out>(loIdx), as DYN_SLICE already does for Queues.
iterateChildren(nodep);
AstNodeExpr* const fromp = nodep->fromp()->unlinkFrBack();
AstConst* const lop = new AstConst{nodep->fileline(), AstConst::WidthedValue{}, 32,
static_cast<uint32_t>(nodep->declRange().lo())};
AstCMethodHard* const newp
= new AstCMethodHard{nodep->fileline(), fromp, VCMethod::ARRAY_SLICE};
newp->addPinsp(lop);
newp->dtypeFrom(nodep); // Reuse the already-correct sliced array dtype
newp->didWidth(true);
newp->protect(false);
nodep->replaceWith(newp);
VL_DO_DANGLING(pushDeletep(nodep), nodep);
}
void visit(AstNode* nodep) override { iterateChildren(nodep); }
public: