Support of wire declaration for the instance ports.#6
Support of wire declaration for the instance ports.#6KyleJeong wants to merge 24 commits intoantoinemadec:masterfrom
Conversation
fix(help): remove duplicated tags
|
Could you please apply the modifications i required in the issue? |
|
Here is what needs to be done:
|
|
OK. How about this way. I will start to modify verilog_instance.py to support wire declaration with the assumption that there could be argv[2]. If argv[2] ==1, wire declaration., else port mapping. "ggb" is OK to me. After that, we can discuss the details about README and help. |
|
I finished the merge (removing verilog_wire.py), if !hasmapto('VerilogInstance') && maparg('gb','n') ==# '' if !hasmapto('VerilogWire') && maparg('ggb','n') ==# '' |
|
@KyleJeong , I will take a look at it in a few days, don't be surprised if my responses are delayed. |
|
Oops, sorry, i completely forgot about this PR. |
before (assume that user copied the port declaration to top)
input clk,
input [2:0] in_a,
input [2:0] in_b,
output reg [3:0] out_sum
module module_name (
input clk,
input [2:0] in_a,
input [2:0] in_b,
output reg [3:0] out_sum
);
...
end module
after (assume the user execute the wire declaration on top and execute the instantiation on buttom)
wire clk;
wire [2:0] in_a;
wire [2:0] in_b;
wire [3:0] out_sum;
module module_name (
.clk(clk),
.in_a(in_a),
.in_b(in_b),
.out_sum(out_sum)
);
...
end module
If we can do that in one command, it will be better definetely.