[vpr][CLI] remove generate net timing from CLI parameters and generate the report by default

Author: amin1377

vpr/src/base/SetupVPR.cpp

@@ -715,7 +715,6 @@ static void SetupAnalysisOpts(const t_options& Options, t_analysis_opts& analysi
 
     analysis_opts.timing_update_type = Options.timing_update_type;
     analysis_opts.write_timing_summary = Options.write_timing_summary;
-    analysis_opts.generate_net_timing_report = Options.generate_net_timing_report;
 }
 
 static void SetupPowerOpts(const t_options& Options, t_power_opts* power_opts, t_arch* Arch) {

vpr/src/base/read_options.cpp

@@ -3011,11 +3011,6 @@ argparse::ArgumentParser create_arg_parser(const std::string& prog_name, t_optio
         .help("Writes implemented design final timing summary to the specified JSON, XML or TXT file.")
         .show_in(argparse::ShowIn::HELP_ONLY);
 
-    analysis_grp.add_argument<bool, ParseOnOff>(args.generate_net_timing_report, "--generate_net_timing_report")
-        .help("Generates a net timing report for each net in the design.")
-        .default_value("off")
-        .show_in(argparse::ShowIn::HELP_ONLY);
-
     auto& power_grp = parser.add_argument_group("power analysis options");
 
     power_grp.add_argument<bool, ParseOnOff>(args.do_power, "--power")

vpr/src/base/read_options.h

@@ -267,7 +267,6 @@ struct t_options {
     argparse::ArgValue<e_post_synth_netlist_unconn_handling> post_synth_netlist_unconn_output_handling;
     argparse::ArgValue<bool> post_synth_netlist_module_parameters;
     argparse::ArgValue<std::string> write_timing_summary;
-    argparse::ArgValue<bool> generate_net_timing_report;
 };
 
 argparse::ArgumentParser create_arg_parser(const std::string& prog_name, t_options& args);

vpr/src/base/vpr_api.cpp

@@ -1464,6 +1464,7 @@ void vpr_analysis(const Netlist<>& net_list,
                                    vpr_setup.AnalysisOpts, vpr_setup.RouterOpts.flat_routing, blk_loc_registry);
         generate_setup_timing_stats(/*prefix=*/"", *timing_info, *analysis_delay_calc,
                                     vpr_setup.AnalysisOpts, vpr_setup.RouterOpts.flat_routing, blk_loc_registry);
+        generate_net_timing_report(/*prefix=*/"", *timing_info, *analysis_delay_calc);
 
         //Write the post-synthesis netlist
         if (vpr_setup.AnalysisOpts.gen_post_synthesis_netlist) {
@@ -1476,10 +1477,6 @@ void vpr_analysis(const Netlist<>& net_list,
             merged_netlist_writer(atom_ctx.netlist().netlist_name(), analysis_delay_calc, Arch.models, vpr_setup.AnalysisOpts);
         }
 
-        if (vpr_setup.AnalysisOpts.generate_net_timing_report) {
-            generate_net_timing_report(/*prefix=*/"", *timing_info, *analysis_delay_calc);
-        }
-
         //Do power analysis
         // TODO: Still assumes that cluster net list is used
         if (vpr_setup.PowerOpts.do_power) {

vpr/src/base/vpr_types.h

@@ -1285,7 +1285,6 @@ struct t_analysis_opts {
     bool timing_report_skew;
     std::string echo_dot_timing_graph_node;
     std::string write_timing_summary;
-    bool generate_net_timing_report;
 
     e_timing_update_type timing_update_type;
 };