check the refomatted code in 57 files

Author: soheilshahrouz

odin_ii/src/ast/ast_loop_unroll.cpp

@@ -342,7 +342,6 @@ post_condition_function resolve_binary_operation(ast_node_t* node) {
     }
 }
 
-
 /*
  *  (function: resolve_post_condition)
  *  return a lambda which gives the next value

vpr/src/analytical_place/analytical_placement_flow.cpp

@@ -68,7 +68,7 @@ static void print_ap_netlist_stats(const APNetlist& netlist) {
  *  @param p_placement            The partial placement to be updated which is assumend
  * to be generated on ap_netlist or have the same blocks.
  */
-static void convert_flat_to_partial_placement(const FlatPlacementInfo& flat_placement_info, const APNetlist& ap_netlist, const Prepacker& prepacker, PartialPlacement& p_placement){
+static void convert_flat_to_partial_placement(const FlatPlacementInfo& flat_placement_info, const APNetlist& ap_netlist, const Prepacker& prepacker, PartialPlacement& p_placement) {
     for (APBlockId ap_blk_id : ap_netlist.blocks()) {
         // Get the molecule that AP block represents
         PackMoleculeId mol_id = ap_netlist.block_molecule(ap_blk_id);
@@ -78,7 +78,7 @@ static void convert_flat_to_partial_placement(const FlatPlacementInfo& flat_plac
         float atom_loc_x, atom_loc_y, atom_loc_layer;
         int atom_loc_sub_tile;
         bool found_valid_atom = false;
-        for (AtomBlockId atom_blk_id: mol.atom_block_ids) {
+        for (AtomBlockId atom_blk_id : mol.atom_block_ids) {
             if (!atom_blk_id.is_valid())
                 continue;
             float current_loc_x = flat_placement_info.blk_x_pos[atom_blk_id];
@@ -87,11 +87,12 @@ static void convert_flat_to_partial_placement(const FlatPlacementInfo& flat_plac
             int current_loc_sub_tile = flat_placement_info.blk_sub_tile[atom_blk_id];
             if (found_valid_atom) {
                 if (current_loc_x != atom_loc_x || current_loc_y != atom_loc_y || current_loc_layer != atom_loc_layer || current_loc_sub_tile != atom_loc_sub_tile)
-                    VPR_FATAL_ERROR(VPR_ERROR_AP, "Molecule of ID %zu contains atom %s (ID: %zu) with a location (%g, %g, layer: %g, subtile: %d) "
-                                                  "that conflicts the location of other atoms in this molecule of (%g, %g, layer: %g, subtile: %d).",
-                                                  mol_id, g_vpr_ctx.atom().netlist().block_name(atom_blk_id).c_str(), atom_blk_id,
-                                                  current_loc_x, current_loc_y, current_loc_layer, current_loc_sub_tile,
-                                                  atom_loc_x, atom_loc_y, atom_loc_layer, atom_loc_sub_tile);
+                    VPR_FATAL_ERROR(VPR_ERROR_AP,
+                                    "Molecule of ID %zu contains atom %s (ID: %zu) with a location (%g, %g, layer: %g, subtile: %d) "
+                                    "that conflicts the location of other atoms in this molecule of (%g, %g, layer: %g, subtile: %d).",
+                                    mol_id, g_vpr_ctx.atom().netlist().block_name(atom_blk_id).c_str(), atom_blk_id,
+                                    current_loc_x, current_loc_y, current_loc_layer, current_loc_sub_tile,
+                                    atom_loc_x, atom_loc_y, atom_loc_layer, atom_loc_sub_tile);
             } else {
                 atom_loc_x = current_loc_x;
                 atom_loc_y = current_loc_y;
@@ -201,4 +202,3 @@ void run_analytical_placement_flow(t_vpr_setup& vpr_setup) {
     g_vpr_ctx.mutable_placement().clean_placement_context_post_place();
     g_vpr_ctx.mutable_floorplanning().clean_floorplanning_context_post_place();
 }
-

vpr/src/analytical_place/analytical_placement_flow.h

@@ -16,4 +16,3 @@ struct t_vpr_setup;
  *  @param vpr_setup    The setup options provided by the user.
  */
 void run_analytical_placement_flow(t_vpr_setup& vpr_setup);
-

vpr/src/analytical_place/analytical_solver.cpp

@@ -57,9 +57,9 @@ std::unique_ptr<AnalyticalSolver> make_analytical_solver(e_analytical_solver sol
 }
 
 AnalyticalSolver::AnalyticalSolver(const APNetlist& netlist)
-    : netlist_(netlist),
-      blk_id_to_row_id_(netlist.blocks().size(), APRowId::INVALID()),
-      row_id_to_blk_id_(netlist.blocks().size(), APBlockId::INVALID()) {
+    : netlist_(netlist)
+    , blk_id_to_row_id_(netlist.blocks().size(), APRowId::INVALID())
+    , row_id_to_blk_id_(netlist.blocks().size(), APBlockId::INVALID()) {
     // Get the number of moveable blocks in the netlist and create a unique
     // row ID from [0, num_moveable_blocks) for each moveable block in the
     // netlist.
@@ -261,16 +261,15 @@ void QPHybridSolver::init_linear_system() {
  *  @param row_id_to_blk_id     Lookup for the row id from the APBlock Id.
  *  @param iteration        The current iteration of the Global Placer.
  */
-static inline void update_linear_system_with_anchors(
-                               Eigen::SparseMatrix<double> &A_sparse_diff,
-                               Eigen::VectorXd &b_x_diff,
-                               Eigen::VectorXd &b_y_diff,
-                               PartialPlacement& p_placement,
-                               size_t num_moveable_blocks,
-                               vtr::vector<APRowId, APBlockId> row_id_to_blk_id,
-                               unsigned iteration) {
+static inline void update_linear_system_with_anchors(Eigen::SparseMatrix<double>& A_sparse_diff,
+                                                     Eigen::VectorXd& b_x_diff,
+                                                     Eigen::VectorXd& b_y_diff,
+                                                     PartialPlacement& p_placement,
+                                                     size_t num_moveable_blocks,
+                                                     vtr::vector<APRowId, APBlockId> row_id_to_blk_id,
+                                                        unsigned iteration) {
     // Anchor weights grow exponentially with iteration.
-    double coeff_pseudo_anchor = 0.01 * std::exp((double)iteration/5);
+    double coeff_pseudo_anchor = 0.01 * std::exp((double)iteration / 5);
     for (size_t row_id_idx = 0; row_id_idx < num_moveable_blocks; row_id_idx++) {
         APRowId row_id = APRowId(row_id_idx);
         APBlockId blk_id = row_id_to_blk_id[row_id];
@@ -281,7 +280,7 @@ static inline void update_linear_system_with_anchors(
     }
 }
 
-void QPHybridSolver::solve(unsigned iteration, PartialPlacement &p_placement) {
+void QPHybridSolver::solve(unsigned iteration, PartialPlacement& p_placement) {
     // Create a temporary linear system which will contain the original linear
     // system which may be updated to include the anchor points.
     Eigen::SparseMatrix<double> A_sparse_diff = Eigen::SparseMatrix<double>(A_sparse);
@@ -304,7 +303,7 @@ void QPHybridSolver::solve(unsigned iteration, PartialPlacement &p_placement) {
     //  - This tolerance may need to be a function of the number of nets.
     //  - Instead of normalizing the fixed blocks, the tolerance can be scaled
     //    by the size of the device.
-    Eigen::ConjugateGradient<Eigen::SparseMatrix<double>, Eigen::Lower|Eigen::Upper> cg;
+    Eigen::ConjugateGradient<Eigen::SparseMatrix<double>, Eigen::Lower | Eigen::Upper> cg;
     cg.compute(A_sparse_diff);
     VTR_ASSERT(cg.info() == Eigen::Success && "Conjugate Gradient failed at compute!");
     // Use the solver to solve for x and y using the constant vectors
@@ -330,4 +329,3 @@ void QPHybridSolver::solve(unsigned iteration, PartialPlacement &p_placement) {
 }
 
 #endif // EIGEN_INSTALLED
-

vpr/src/analytical_place/analytical_solver.h

@@ -24,7 +24,7 @@
 
 // Pop the GCC diagnostics state back to what it was before.
 #pragma GCC diagnostic pop
-#endif  // EIGEN_INSTALLED
+#endif // EIGEN_INSTALLED
 
 // Forward declarations
 class PartialPlacement;
@@ -36,7 +36,7 @@ class APNetlist;
  * NOTE: More are coming.
  */
 enum class e_analytical_solver {
-    QP_HYBRID       // A solver which optimizes the quadratic HPWL of the design.
+    QP_HYBRID // A solver which optimizes the quadratic HPWL of the design.
 };
 
 /**
@@ -58,7 +58,7 @@ typedef vtr::StrongId<ap_row_id_tag, size_t> APRowId;
  * compare different solvers.
  */
 class AnalyticalSolver {
-public:
+  public:
     virtual ~AnalyticalSolver() {}
 
     /**
@@ -67,7 +67,7 @@ class AnalyticalSolver {
      * Initializes the internal data members of the base class which are useful
      * for all solvers.
      */
-    AnalyticalSolver(const APNetlist &netlist);
+    AnalyticalSolver(const APNetlist& netlist);
 
     /**
      * @brief Run an iteration of the solver using the given partial placement
@@ -87,10 +87,9 @@ class AnalyticalSolver {
      *  @param p_placement  A "hint" to a legal solution that the solver should
      *                      try and be like.
      */
-    virtual void solve(unsigned iteration, PartialPlacement &p_placement) = 0;
-
-protected:
+    virtual void solve(unsigned iteration, PartialPlacement& p_placement) = 0;
 
+  protected:
     /// @brief The APNetlist the solver is optimizing over. It is implied that
     ///        the netlist is not being modified during global placement.
     const APNetlist& netlist_;
@@ -115,7 +114,7 @@ class AnalyticalSolver {
  * @brief A factory method which creates an Analytical Solver of the given type.
  */
 std::unique_ptr<AnalyticalSolver> make_analytical_solver(e_analytical_solver solver_type,
-                                                         const APNetlist &netlist);
+                                                         const APNetlist& netlist);
 
 // The Eigen library is used to solve matrix equations in the following solvers.
 // The solver cannot be built if Eigen is not installed.
@@ -145,7 +144,7 @@ std::unique_ptr<AnalyticalSolver> make_analytical_solver(e_analytical_solver sol
  *          https://doi.org/10.1109/TCAD.2005.846365
  */
 class QPHybridSolver : public AnalyticalSolver {
-private:
+  private:
     /// @brief The threshold for the number of pins a net will have to use the
     ///        Star or Clique net models. If the number of pins is larger
     ///        than this number, a star node will be created.
@@ -182,14 +181,14 @@ class QPHybridSolver : public AnalyticalSolver {
     /// @brief The constant vector in the y dimension for the linear system.
     Eigen::VectorXd b_y;
 
-public:
-
+  public:
     /**
      * @brief Constructor of the QPHybridSolver
      *
      * Initializes internal data and constructs the initial linear system.
      */
-    QPHybridSolver(const APNetlist& netlist) : AnalyticalSolver(netlist) {
+    QPHybridSolver(const APNetlist& netlist)
+        : AnalyticalSolver(netlist) {
         // Initializing the linear system only depends on the netlist and fixed
         // block locations. Both are provided by the netlist, allowing this to
         // be initialized in the constructor.
@@ -216,8 +215,7 @@ class QPHybridSolver : public AnalyticalSolver {
      *  @param p_placement  A "guess" solution. The result will be written into
      *                      this object.
      */
-    void solve(unsigned iteration, PartialPlacement &p_placement) final;
+    void solve(unsigned iteration, PartialPlacement& p_placement) final;
 };
 
 #endif // EIGEN_INSTALLED
-

vpr/src/analytical_place/ap_flow_enums.h

@@ -26,8 +26,8 @@ enum class e_ap_global_placer {
  * enum can select between these different Full Legalizers.
  */
 enum class e_ap_full_legalizer {
-    Naive,  ///< The Naive Full Legalizer, which clusters atoms placed in the same tile and tries to place them in that tile according to the flat placement.
-    APPack,  ///< The APPack Full Legalizer, which uses the flat placement to improve the Packer and Placer.
+    Naive,                ///< The Naive Full Legalizer, which clusters atoms placed in the same tile and tries to place them in that tile according to the flat placement.
+    APPack,               ///< The APPack Full Legalizer, which uses the flat placement to improve the Packer and Placer.
     Basic_Min_Disturbance ///< The Basic Min. Disturbance Full Legalizer, which tries to reconstruct a clustered placement that is as close to the incoming flat placement as it can.
 };
 
@@ -38,7 +38,6 @@ enum class e_ap_full_legalizer {
  * enum can select between these different Detailed Placers.
  */
 enum class e_ap_detailed_placer {
-    Identity,           ///< The Identity Detailed Placer, which does not perform any optimizations on the legalized placement. Needed as a placeholder.
-    Annealer            ///< The Annealer Detailed Placer, which runs the annealer found in the Place part of the VPR flow (using the same options as the Placement stage).
+    Identity, ///< The Identity Detailed Placer, which does not perform any optimizations on the legalized placement. Needed as a placeholder.
+    Annealer  ///< The Annealer Detailed Placer, which runs the annealer found in the Place part of the VPR flow (using the same options as the Placement stage).
 };
-

vpr/src/analytical_place/ap_netlist.h

@@ -49,8 +49,8 @@ struct APFixedBlockLoc {
  *       block ids.
  */
 enum class APBlockMobility : bool {
-    MOVEABLE,   // The block is not constrained in any dimension.
-    FIXED       // The block is fixed.
+    MOVEABLE, // The block is not constrained in any dimension.
+    FIXED     // The block is fixed.
 };
 
 /**
@@ -62,20 +62,21 @@ enum class APBlockMobility : bool {
  * APBlocks. These need not have physical meaning.
  */
 class APNetlist : public Netlist<APBlockId, APPortId, APPinId, APNetId> {
-public:
+  public:
     /**
      * @brief Constructs a netlist
      *
      *  @param name The name of the netlist (e.g. top-level module)
      *  @param id   A unique identifier for the netlist (e.g. a secure digest of
      *              the input file)
      */
-    APNetlist(std::string name = "", std::string id = "") : Netlist(name, id) {}
+    APNetlist(std::string name = "", std::string id = "")
+        : Netlist(name, id) {}
 
     APNetlist(const APNetlist& rhs) = default;
     APNetlist& operator=(const APNetlist& rhs) = default;
 
-public: // Public Accessors
+  public: // Public Accessors
     /*
      * Blocks
      */
@@ -90,7 +91,7 @@ class APNetlist : public Netlist<APBlockId, APPortId, APPinId, APNetId> {
     ///        This method should not be used if the block is moveable.
     const APFixedBlockLoc& block_loc(const APBlockId id) const;
 
-public: // Public Mutators
+  public: // Public Mutators
     /*
      * Note: all create_*() functions will silently return the appropriate ID
      * if it has already been created.
@@ -141,7 +142,7 @@ class APNetlist : public Netlist<APBlockId, APPortId, APPinId, APNetId> {
      */
     APNetId create_net(const std::string& name);
 
-private: // Private Members
+  private: // Private Members
     /*
      * Netlist compression / optimization
      */
@@ -178,7 +179,7 @@ class APNetlist : public Netlist<APBlockId, APPortId, APPinId, APNetId> {
     bool validate_pin_sizes_impl(size_t num_pins) const override;
     bool validate_net_sizes_impl(size_t num_nets) const override;
 
-private: // Private Data
+  private: // Private Data
     /// @brief Molecule of each block
     vtr::vector_map<APBlockId, PackMoleculeId> block_molecules_;
     /// @brief Type of each block
@@ -187,4 +188,3 @@ class APNetlist : public Netlist<APBlockId, APPortId, APPinId, APNetId> {
     ///        NOTE: This vector will likely be quite sparse.
     vtr::vector_map<APBlockId, APFixedBlockLoc> block_locs_;
 };
-

vpr/src/analytical_place/ap_netlist_fwd.h

@@ -26,7 +26,7 @@ class APNetlist;
  * @brief A unique identifier for a block in the AP netlist
  */
 class APBlockId : public ParentBlockId {
-public:
+  public:
     static constexpr APBlockId INVALID() { return APBlockId(); }
     using ParentBlockId::ParentBlockId;
 
@@ -37,7 +37,7 @@ class APBlockId : public ParentBlockId {
  * @brief A unique identifier for a net in the AP netlist
  */
 class APNetId : public ParentNetId {
-public:
+  public:
     static constexpr APNetId INVALID() { return APNetId(); }
     using ParentNetId::ParentNetId;
 
@@ -48,7 +48,7 @@ class APNetId : public ParentNetId {
  * @brief A unique identifier for a port in the AP netlist
  */
 class APPortId : public ParentPortId {
-public:
+  public:
     static constexpr APPortId INVALID() { return APPortId(); }
     using ParentPortId::ParentPortId;
 
@@ -59,7 +59,7 @@ class APPortId : public ParentPortId {
  * @brief A unique identifier for a pin in the AP netlist
  */
 class APPinId : public ParentPinId {
-public:
+  public:
     static constexpr APPinId INVALID() { return APPinId(); }
     using ParentPinId::ParentPinId;
 
@@ -101,4 +101,3 @@ struct hash<APPinId> {
     }
 };
 } // namespace std
-

vpr/src/analytical_place/flat_placement_density_manager.h

@@ -57,7 +57,7 @@ struct t_physical_tile_type;
  *       may allow us to trade off quality and runtime.
  */
 class FlatPlacementDensityManager {
-public:
+  public:
     /**
      * @brief Construct the density manager.
      *
@@ -233,7 +233,7 @@ class FlatPlacementDensityManager {
      */
     void print_bin_grid() const;
 
-private:
+  private:
     /// @brief The AP netlist of blocks which are filling the bins.
     const APNetlist& ap_netlist_;
 
@@ -268,4 +268,3 @@ class FlatPlacementDensityManager {
     /// @brief The verbosity of log messages in this class.
     const int log_verbosity_;
 };
-

vpr/src/analytical_place/flat_placement_mass_calculator.h

@@ -39,7 +39,7 @@ struct t_physical_tile_type;
  * mass that the tile should aim to hold.
  */
 class FlatPlacementMassCalculator {
-public:
+  public:
     /**
      * @brief Construct the mass calculator.
      *
@@ -97,7 +97,7 @@ class FlatPlacementMassCalculator {
         return block_mass_[blk_id];
     }
 
-private:
+  private:
     /// @brief The capacity of each physical tile type, indexed by the index
     ///        of the physical_tile_type.
     std::vector<PrimitiveVector> physical_tile_type_capacity_;
@@ -112,4 +112,3 @@ class FlatPlacementMassCalculator {
     /// @brief The verbosity of log messages in the mass calculator.
     const int log_verbosity_;
 };
-