Skip to content

Commit 0bec813

Browse files
committed
removed unnecessary function parameters from the remove_one_lut_nodes function
1 parent 5f98b6b commit 0bec813

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

utils/vqm2blif/src/base/cleanup.cpp

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ void build_netlist (t_module* module, busvec* buses, s_hash** hash_table);
1212
void init_nets (t_pin_def** pins, int num_pins, busvec* buses, struct s_hash** hash_table);
1313
void set_net_assigns (t_assign** assignments, int num_assigns, busvec* buses, struct s_hash** hash_table);
1414
void add_subckts (t_node** nodes, int num_nodes, busvec* buses, struct s_hash** hash_table);
15-
void remove_one_lut_nodes ( busvec* buses, struct s_hash** hash_table, t_node** nodes, int original_num_nodes, t_module* module );
15+
void remove_one_lut_nodes ( busvec* buses, struct s_hash** hash_table, t_module* module );
1616
void clean_netlist ( busvec* buses, struct s_hash** hash_table, t_node** nodes, int num_nodes );
1717
void reassign_net_source (t_net* net);
1818
void print_to_module ( t_module* module, busvec* buses, struct s_hash** hash_table );
@@ -49,7 +49,7 @@ void netlist_cleanup (t_module* module){
4949

5050
cout << "\t>> Removing One-LUTs" << "...\n";
5151

52-
remove_one_lut_nodes ( &buses, hash_table, module->array_of_nodes, module->number_of_nodes, module );
52+
remove_one_lut_nodes ( &buses, hash_table, module );
5353

5454
cout << "\t>> Removing buffered nets" << ((clean_mode == CL_BUFF)? "":" and inverted subckt inputs") << "...\n";
5555

@@ -237,7 +237,7 @@ void add_subckts (t_node** nodes, int num_nodes, busvec* buses, struct s_hash**
237237
//============================================================================================
238238
//============================================================================================
239239

240-
void remove_one_lut_nodes ( busvec* buses, struct s_hash** hash_table, t_node** nodes, int original_num_nodes, t_module* module ){
240+
void remove_one_lut_nodes ( busvec* buses, struct s_hash** hash_table, t_module* module ){
241241
/*
242242
Quartus fitter may have introduced some one-LUTs in the post-fit netlist that makes it harder for VPR to place and route.
243243
Generally, these one-LUTs are inserted by the Quartus router in order to pass a signal through a LUT to the FF in the same
@@ -278,11 +278,17 @@ void remove_one_lut_nodes ( busvec* buses, struct s_hash** hash_table, t_node**
278278
|
279279
---->
280280
Change Log:
281+
- Srivatsan Srinivasan, September 2021:
282+
- removed the function parameters "original_num_nodes" and "nodes". These values can be from the "module" parameter and are now assigned internally within this function.
281283
- Srivatsan Srinivasan, August 2021:
282284
- Moved the incrementing of the "oneluts_elim" variable to this fuction from the "remove_node" function. The purpose of this change was to localise any vairable attached to removing one lut nodes within this function. Additionally, now the "remove_node" function is generalized and is not limited to be only used by "remove_one_lut_nodes".
283285
*/
284286
oneluts_elim = 0;
285287

288+
// parameters related to the module
289+
int original_num_nodes = module->number_of_nodes;
290+
t_node** nodes = module->array_of_nodes;
291+
286292
t_node* temp_node;
287293
t_node_port_association* temp_port;
288294
netvec* temp_bus;

0 commit comments

Comments
 (0)