From b760d03da5a87de39b0098bc529fd5112f5a78f4 Mon Sep 17 00:00:00 2001
From: soheilshahrouz <soheilqs@gmail.com>
Date: Sun, 24 Nov 2024 16:38:07 -0500
Subject: [PATCH] quiet the placement timer if the placer object is quiet

---
 libs/libvtrutil/src/vtr_time.h | 2 +-
 vpr/src/place/place.cpp        | 5 +----
 vpr/src/place/placer.cpp       | 2 ++
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/libs/libvtrutil/src/vtr_time.h b/libs/libvtrutil/src/vtr_time.h
index 4e389ef5026..3f187e59288 100644
--- a/libs/libvtrutil/src/vtr_time.h
+++ b/libs/libvtrutil/src/vtr_time.h
@@ -36,7 +36,7 @@ class Timer {
     constexpr static float BYTE_TO_MIB = 1024 * 1024;
 };
 
-///@brief Scoped time class which prints the time elapsed for the specifid action
+///@brief Scoped time class which prints the time elapsed for the specified action
 class ScopedActionTimer : public Timer {
   public:
     ScopedActionTimer(std::string action);
diff --git a/vpr/src/place/place.cpp b/vpr/src/place/place.cpp
index f00a2100a76..86d1f374212 100644
--- a/vpr/src/place/place.cpp
+++ b/vpr/src/place/place.cpp
@@ -1,3 +1,4 @@
+
 #include <memory>
 
 #include "vtr_assert.h"
@@ -47,10 +48,6 @@ void try_place(const Netlist<>& net_list,
                std::vector<t_segment_inf>& segment_inf,
                const std::vector<t_direct_inf>& directs,
                bool is_flat) {
-    /* Does almost all the work of placing a circuit.  Width_fac gives the   *
-     * width of the widest channel.  Place_cost_exp says what exponent the   *
-     * width should be taken to when calculating costs.  This allows a       *
-     * greater bias for anisotropic architectures.                           */
 
     /* Currently, the functions that require is_flat as their parameter and are called during placement should
      * receive is_flat as false. For example, if the RR graph of router lookahead is built here, it should be as
diff --git a/vpr/src/place/placer.cpp b/vpr/src/place/placer.cpp
index 3a56a4e03e5..12dc5276646 100644
--- a/vpr/src/place/placer.cpp
+++ b/vpr/src/place/placer.cpp
@@ -51,6 +51,7 @@ Placer::Placer(const Netlist<>& net_list,
 
     // Start measuring placement time
     timer_ = std::make_unique<vtr::ScopedStartFinishTimer>("Placement");
+    timer_->quiet(quiet);
 
     /* To make sure the importance of NoC-related cost terms compared to
      * BB and timing cost is determine only through NoC placement weighting factor,
@@ -101,6 +102,7 @@ Placer::Placer(const Netlist<>& net_list,
        }
 #endif
 
+       // width_fac gives the width of the widest channel
        const int width_fac = placer_opts.place_chan_width;
        init_draw_coords((float)width_fac, placer_state_.blk_loc_registry());
    }