Support empty queue as dynarray default value (#5055)

Signed-off-by: Arkadiusz Kozdra <akozdra@antmicro.com>
This commit is contained in:
Arkadiusz Kozdra 2024-04-18 18:53:23 +02:00 committed by GitHub
parent 1315aa31ed
commit 5b839699ac
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 52 additions and 6 deletions

View File

@ -1839,10 +1839,13 @@ AstNodeFTask* V3Task::taskConnectWrapNew(AstNodeFTask* taskp, const string& newn
newPortp->funcLocal(true); newPortp->funcLocal(true);
newTaskp->addStmtsp(newPortp); newTaskp->addStmtsp(newPortp);
// Runtime-assign it to the default // Runtime-assign it to the default
AstAssign* const newAssignp = new AstAssign{ if (!VN_IS(valuep, EmptyQueue)) {
valuep->fileline(), new AstVarRef{valuep->fileline(), newPortp, VAccess::WRITE}, AstAssign* const newAssignp
valuep->cloneTree(true)}; = new AstAssign{valuep->fileline(),
newTaskp->addStmtsp(newAssignp); new AstVarRef{valuep->fileline(), newPortp, VAccess::WRITE},
valuep->cloneTree(true)};
newTaskp->addStmtsp(newAssignp);
}
} }
oldNewVars.emplace(portp, newPortp); oldNewVars.emplace(portp, newPortp);
const VAccess pinAccess = portp->isWritable() ? VAccess::WRITE : VAccess::READ; const VAccess pinAccess = portp->isWritable() ? VAccess::WRITE : VAccess::READ;

View File

@ -1228,7 +1228,7 @@ class WidthVisitor final : public VNVisitor {
} }
void visit(AstEmptyQueue* nodep) override { void visit(AstEmptyQueue* nodep) override {
nodep->dtypeSetEmptyQueue(); nodep->dtypeSetEmptyQueue();
if (!VN_IS(nodep->backp(), Assign)) { if (!VN_IS(nodep->backp(), Assign) && !VN_IS(nodep->backp(), Var)) {
nodep->v3warn(E_UNSUPPORTED, nodep->v3warn(E_UNSUPPORTED,
"Unsupported/Illegal: empty queue ('{}') in this context"); "Unsupported/Illegal: empty queue ('{}') in this context");
} }
@ -4888,7 +4888,8 @@ class WidthVisitor final : public VNVisitor {
} }
if (VN_IS(nodep->rhsp(), EmptyQueue)) { if (VN_IS(nodep->rhsp(), EmptyQueue)) {
UINFO(9, "= {} -> .delete(): " << nodep); UINFO(9, "= {} -> .delete(): " << nodep);
if (!VN_IS(nodep->lhsp()->dtypep()->skipRefp(), QueueDType)) { const AstNodeDType* const lhsDtp = nodep->lhsp()->dtypep()->skipRefp();
if (!VN_IS(lhsDtp, QueueDType) && !VN_IS(lhsDtp, DynArrayDType)) {
nodep->v3warn(E_UNSUPPORTED, nodep->v3warn(E_UNSUPPORTED,
"Unsupported/Illegal: empty queue ('{}') in this assign context"); "Unsupported/Illegal: empty queue ('{}') in this assign context");
VL_DO_DANGLING(pushDeletep(nodep->unlinkFrBack()), nodep); VL_DO_DANGLING(pushDeletep(nodep->unlinkFrBack()), nodep);

View File

@ -13,6 +13,7 @@ module t (/*AUTOARG*/);
int a2[] = {14, 15}; int a2[] = {14, 15};
int a3[] = '{16}; int a3[] = '{16};
int a4[] = {17}; int a4[] = {17};
int a5[] = {};
initial begin initial begin
`checkh(a1.size, 2); `checkh(a1.size, 2);
@ -29,6 +30,8 @@ module t (/*AUTOARG*/);
`checkh(a4.size, 1); `checkh(a4.size, 1);
`checkh(a4[0], 17); `checkh(a4[0], 17);
`checkh(a5.size, 0);
$write("*-* All Finished *-*\n"); $write("*-* All Finished *-*\n");
$finish; $finish;
end end

View File

@ -0,0 +1,21 @@
#!/usr/bin/env perl
if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); die; }
# DESCRIPTION: Verilator: Verilog Test driver/expect definition
#
# Copyright 2022 by Wilson Snyder. This program is free software; you
# can redistribute it and/or modify it under the terms of either the GNU
# Lesser General Public License Version 3 or the Perl Artistic License
# Version 2.0.
# SPDX-License-Identifier: LGPL-3.0-only OR Artistic-2.0
scenarios(simulator => 1);
compile(
);
execute(
check_finished => 1,
);
ok(1);
1;

View File

@ -0,0 +1,18 @@
// DESCRIPTION: Verilator: Verilog Test module
//
// This file ONLY is placed under the Creative Commons Public Domain, for
// any use, without warranty, 2024 by Antmicro Ltd.
// SPDX-License-Identifier: CC0-1.0
module t (/*AUTOARG*/);
task tsk(int q[] = {});
if (q.size != 0) $stop;
endtask
initial begin
tsk();
$write("*-* All Finished *-*\n");
$finish;
end
endmodule