Skip to content

Commit 5dc8e83

Browse files
committed
address review comments
Signed-off-by: Alessandro Comodi <[email protected]>
1 parent c0d896a commit 5dc8e83

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

vpr/src/base/netlist_writer.cpp

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,9 @@
9292
//File local type declarations
9393
//
9494

95+
// This pair cointains the following values:
96+
// - double: hold, setup or clock-to-q delays of the port
97+
// - string: port name of the associated source clock pin of the sequential port
9598
typedef std::pair<double, std::string> sequential_port_delay_pair;
9699

97100
/*enum class PortType {
@@ -1622,6 +1625,14 @@ class NetlistWriterVisitor : public NetlistVisitor {
16221625
std::map<std::string, std::vector<std::string>> input_port_conns;
16231626
std::map<std::string, std::vector<std::string>> output_port_conns;
16241627
std::vector<Arc> timing_arcs;
1628+
1629+
// Maps to store a sink's port with the corresponding timing edge to that sink
1630+
// - key : string corresponding to the port's name
1631+
// - value : pair with the delay and the associated clock pin port name
1632+
//
1633+
// tsu : Setup
1634+
// thld: Hold
1635+
// tcq : Clock-to-Q
16251636
std::map<std::string, sequential_port_delay_pair> ports_tsu;
16261637
std::map<std::string, sequential_port_delay_pair> ports_thld;
16271638
std::map<std::string, sequential_port_delay_pair> ports_tcq;
@@ -1651,7 +1662,7 @@ class NetlistWriterVisitor : public NetlistVisitor {
16511662
net = make_inst_wire(atom_net_id, src_tnode, inst_name, PortType::INPUT, iport, ipin);
16521663
//Delays
16531664
//
1654-
//We record the souce sink tnodes and thier delays here
1665+
//We record the source's sink tnodes and their delays here
16551666
for (tatum::EdgeId edge : timing_ctx.graph->node_out_edges(src_tnode)) {
16561667
double delay = delay_calc_->max_edge_delay(*timing_ctx.graph, edge);
16571668

0 commit comments

Comments
 (0)