Merge pull request #1557 from Bill-hbrhbr/FixMemberInitialization

Add in-class initializers for t_placer_costs to fix coverity scan defect

Author: Bill-hbrhbr

vpr/src/place/place_util.cpp

@@ -278,6 +278,7 @@ void t_annealing_state::update_move_lim(float success_target, float success_rate
     move_lim = std::max(move_lim, 1);
 }
 
+///@brief Clear all data fields.
 void t_placer_statistics::reset() {
     av_cost = 0.;
     av_bb_cost = 0.;
@@ -288,6 +289,7 @@ void t_placer_statistics::reset() {
     std_dev = 0.;
 }
 
+///@brief Calculate placer success rate and cost std_dev for this iteration.
 void t_placer_statistics::single_swap_update(const t_placer_costs& costs) {
     success_sum++;
     av_cost += costs.cost;
@@ -296,6 +298,7 @@ void t_placer_statistics::single_swap_update(const t_placer_costs& costs) {
     sum_of_squares += (costs.cost) * (costs.cost);
 }
 
+///@brief Update stats when a single swap move has been accepted.
 void t_placer_statistics::calc_iteration_stats(const t_placer_costs& costs, int move_lim) {
     if (success_sum == 0) {
         av_cost = costs.cost;

vpr/src/place/place_util.h

@@ -40,11 +40,11 @@
  */
 class t_placer_costs {
   public: //members
-    double cost;
-    double bb_cost;
-    double timing_cost;
-    double bb_cost_norm;
-    double timing_cost_norm;
+    double cost = 0.;
+    double bb_cost = 0.;
+    double timing_cost = 0.;
+    double bb_cost_norm = 0.;
+    double timing_cost_norm = 0.;
 
   public: //Constructor
     t_placer_costs(t_place_algorithm algo)