Changed the precision of the noc constraints (noc link bandwidth, link latency, etc..) from integer to double

Author: Srivat97

libs/libarchfpga/src/physical_types.h

@@ -1746,9 +1746,9 @@ struct t_router {
 /* Network-on-chip(NoC) data type used to store the network 
 properties and used when builidng a dedicated on-chip network*/
 struct t_noc_inf {
-    int link_bandwidth; // in Gbps
-    int link_latency; // in nanoseconds
-    int router_latency; // in nanoseconds
+    double link_bandwidth; // in Gbps
+    double link_latency; // in nanoseconds
+    double router_latency; // in nanoseconds
 
     std::vector<t_router> router_list;
 

libs/libarchfpga/src/read_xml_arch_file.cpp

@@ -4669,19 +4669,19 @@ static void ProcessNoc(pugi::xml_node noc_tag, t_arch* arch, const pugiutil::loc
     pugiutil::expect_only_attributes(noc_tag, expected_noc_attributes, loc_data);
 
     // now go through and parse the required attributes for noc tag
-    noc_ref->link_bandwidth = pugiutil::get_attribute(noc_tag, "link_bandwidth", loc_data, pugiutil::REQUIRED).as_int(attribute_conversion_failure);
+    noc_ref->link_bandwidth = pugiutil::get_attribute(noc_tag, "link_bandwidth", loc_data, pugiutil::REQUIRED).as_double(attribute_conversion_failure);
 
-    noc_ref->link_latency = pugiutil::get_attribute(noc_tag, "link_latency", loc_data, pugiutil::REQUIRED).as_int(attribute_conversion_failure);
+    noc_ref->link_latency = pugiutil::get_attribute(noc_tag, "link_latency", loc_data, pugiutil::REQUIRED).as_double(attribute_conversion_failure);
 
-    noc_ref->router_latency = pugiutil::get_attribute(noc_tag, "router_latency", loc_data, pugiutil::REQUIRED).as_int(attribute_conversion_failure);
+    noc_ref->router_latency = pugiutil::get_attribute(noc_tag, "router_latency", loc_data, pugiutil::REQUIRED).as_double(attribute_conversion_failure);
 
     noc_ref->noc_router_tile_name = pugiutil::get_attribute(noc_tag, "noc_router_tile_name", loc_data, pugiutil::REQUIRED).as_string();
 
     // the noc parameters can only be non-zero positive values
     if ((noc_ref->link_bandwidth < 0) || (noc_ref->link_latency < 0) || (noc_ref->router_latency < 0))
     {
         archfpga_throw(loc_data.filename_c_str(), loc_data.line(noc_tag),
-                               "The link bandwidth, link latency and router latency for the NoC must be a positive non-zero integer.");
+                               "The link bandwidth, link latency and router latency for the NoC must be a positive non-zero value.");
     }
 
     // check that the router tile name was supplied properly

vpr/src/base/setup_noc.cpp

@@ -310,11 +310,11 @@ static void echo_noc(char* file_name)
     fprintf(fp, "NoC Constraints:\n");
     fprintf(fp, "--------------------------------------------------------------\n");
     fprintf(fp, "\n");
-    fprintf(fp, "Maximum NoC Link Bandwidth: %d\n", noc_ctx.noc_link_bandwidth);
+    fprintf(fp, "Maximum NoC Link Bandwidth: %f\n", noc_ctx.noc_link_bandwidth);
     fprintf(fp, "\n");
-    fprintf(fp, "NoC Link Latency: %d\n", noc_ctx.noc_link_latency);
+    fprintf(fp, "NoC Link Latency: %f\n", noc_ctx.noc_link_latency);
     fprintf(fp, "\n");
-    fprintf(fp, "NoC Router Latency: %d\n", noc_ctx.noc_router_latency);
+    fprintf(fp, "NoC Router Latency: %f\n", noc_ctx.noc_router_latency);
     fprintf(fp, "\n");
 
     // print all the routers and their properties

vpr/src/base/vpr_context.h

@@ -431,17 +431,17 @@ struct NocContext : public Context {
     /**
      * @brief Represents the maximum allowed bandwidth for the links in the NoC (in Gbps)
      */
-    int noc_link_bandwidth;
+    double noc_link_bandwidth;
 
     /**
      * @brief Represents the delay expected when going through a link
      */
-    int noc_link_latency;
+    double noc_link_latency;
 
     /**
      * @brief Represents the expected delay when going through a router
      */
-    int noc_router_latency;
+    double noc_router_latency;
 };
 
 /**