[AP][Legalizer] Updated Mass Report Based on PR Comments

Author: AlexandreSinger

vpr/src/analytical_place/ap_mass_report.cpp

@@ -46,7 +46,7 @@ struct PrintingTree {
 };
 
 /**
- * @brief Print the given tree node.
+ * @brief Recursively print the given tree node.
  *
  *  @param node
  *      The node to print.
@@ -55,9 +55,9 @@ struct PrintingTree {
  *  @param prefix
  *      The prefix that all children of this node will print before their names.
  */
-void print_tree_node(const PrintingTreeNode& node,
-                     std::ofstream& os,
-                     const std::string& prefix) {
+void print_tree_node_recur(const PrintingTreeNode& node,
+                           std::ofstream& os,
+                           const std::string& prefix) {
     // Print the name of the node here and start a new line.
     os << node.name << "\n";
 
@@ -71,14 +71,14 @@ void print_tree_node(const PrintingTreeNode& node,
             os << prefix << "├── ";
             // Print the child node and update the prefix for any of its children.
             // This prefix will connect the lines in a good looking way.
-            print_tree_node(node.children[child_idx], os, prefix + "│   ");
+            print_tree_node_recur(node.children[child_idx], os, prefix + "│   ");
         } else {
             // If this is the last child, we print an L shape to signify that
             // there are no further children.
             os << prefix << "└── ";
             // Print the child node, and set the prefix to basically just be
             // an indent.
-            print_tree_node(node.children[child_idx], os, prefix + "    ");
+            print_tree_node_recur(node.children[child_idx], os, prefix + "    ");
         }
     }
 }
@@ -94,7 +94,7 @@ void print_tree_node(const PrintingTreeNode& node,
  *      The output file stream to print to.
  */
 void print_tree(const PrintingTree& tree, std::ofstream& os) {
-    print_tree_node(tree.root, os, "");
+    print_tree_node_recur(tree.root, os, "");
 }
 
 /**

vpr/src/base/ShowSetup.cpp

@@ -657,7 +657,6 @@ static void ShowAnalyticalPlacerOpts(const t_ap_opts& APOpts) {
 
     VTR_LOG("AnalyticalPlacerOpts.ap_timing_tradeoff: %f\n", APOpts.ap_timing_tradeoff);
     VTR_LOG("AnalyticalPlacerOpts.log_verbosity: %d\n", APOpts.log_verbosity);
-    VTR_LOG("AnalyticalPlacerOpts.generate_mass_report: %s\n", APOpts.generate_mass_report ? "true" : "false");
 }
 
 static void ShowNetlistOpts(const t_netlist_opts& NetlistOpts) {