mirrors/yosys
3
0
Fork 0
mirror of https://github.com/YosysHQ/yosys synced 2025-11-04 13:29:12 +00:00
Code Activity

Fix constants bound to single bit arguments (fixes #2383)

Browse source
This commit is contained in:
Zachary Snow 2020-12-05 18:56:18 -07:00
parent 95c6086681
commit 8206546c45
2 changed files with 18 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
frontends/ast/simplify.cc
View file

@ -3337,6 +3337,14 @@ skip_dynamic_range_lvalue_expansion:;
wire->type = AST_LOCALPARAM; wire->type = AST_LOCALPARAM;
wire->attributes.erase(ID::nosync); wire->attributes.erase(ID::nosync);
wire->children.insert(wire->children.begin(), arg->clone()); wire->children.insert(wire->children.begin(), arg->clone());
// args without a range implicitly have width 1
if (wire->children.back()->type != AST_RANGE) {
AstNode* range = new AstNode();
range->type = AST_RANGE;
wire->children.push_back(range);
range->children.push_back(mkconst_int(0, true));
range->children.push_back(mkconst_int(0, true));
}
continue; continue;
} }
AstNode *wire_id = new AstNode(AST_IDENTIFIER); AstNode *wire_id = new AstNode(AST_IDENTIFIER);

10
tests/various/const_arg_loop.v
View file

@ -39,6 +39,12 @@ module top;
end end
endfunction endfunction
function automatic [16:0] operation4;
input [15:0] a;
input b;
operation4 = {a, b};
endfunction
wire [31:0] a; wire [31:0] a;
assign a = 2; assign a = 2;
@ -53,6 +59,9 @@ module top;
wire [31:0] x3; wire [31:0] x3;
assign x3 = operation3(A, a); assign x3 = operation3(A, a);
wire [16:0] x4;
assign x4 = operation4(a[15:0], 0);
// `define VERIFY // `define VERIFY
`ifdef VERIFY `ifdef VERIFY
assert property (a == 2); assert property (a == 2);
@ -60,5 +69,6 @@ module top;
assert property (x1 == 16); assert property (x1 == 16);
assert property (x2 == 4); assert property (x2 == 4);
assert property (x3 == 16); assert property (x3 == 16);
assert property (x4 == a << 1);
`endif `endif
endmodule endmodule