Skip to content

Commit b760d03

Browse files
quiet the placement timer if the placer object is quiet
1 parent 877fd8e commit b760d03

File tree

3 files changed

+4
-5
lines changed

3 files changed

+4
-5
lines changed

libs/libvtrutil/src/vtr_time.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ class Timer {
3636
constexpr static float BYTE_TO_MIB = 1024 * 1024;
3737
};
3838

39-
///@brief Scoped time class which prints the time elapsed for the specifid action
39+
///@brief Scoped time class which prints the time elapsed for the specified action
4040
class ScopedActionTimer : public Timer {
4141
public:
4242
ScopedActionTimer(std::string action);

vpr/src/place/place.cpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
12
#include <memory>
23

34
#include "vtr_assert.h"
@@ -47,10 +48,6 @@ void try_place(const Netlist<>& net_list,
4748
std::vector<t_segment_inf>& segment_inf,
4849
const std::vector<t_direct_inf>& directs,
4950
bool is_flat) {
50-
/* Does almost all the work of placing a circuit. Width_fac gives the *
51-
* width of the widest channel. Place_cost_exp says what exponent the *
52-
* width should be taken to when calculating costs. This allows a *
53-
* greater bias for anisotropic architectures. */
5451

5552
/* Currently, the functions that require is_flat as their parameter and are called during placement should
5653
* receive is_flat as false. For example, if the RR graph of router lookahead is built here, it should be as

vpr/src/place/placer.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ Placer::Placer(const Netlist<>& net_list,
5151

5252
// Start measuring placement time
5353
timer_ = std::make_unique<vtr::ScopedStartFinishTimer>("Placement");
54+
timer_->quiet(quiet);
5455

5556
/* To make sure the importance of NoC-related cost terms compared to
5657
* BB and timing cost is determine only through NoC placement weighting factor,
@@ -101,6 +102,7 @@ Placer::Placer(const Netlist<>& net_list,
101102
}
102103
#endif
103104

105+
// width_fac gives the width of the widest channel
104106
const int width_fac = placer_opts.place_chan_width;
105107
init_draw_coords((float)width_fac, placer_state_.blk_loc_registry());
106108
}

0 commit comments

Comments
 (0)