added directives to the draw noc related code so that there is no build failure when graphics isn't enabled

Author: Srivat97

vpr/src/base/noc_data_types.h

@@ -3,7 +3,7 @@
 
 #include "vtr_strong_id.h"
 
-// definitins data types used to index the routers and links within the noc
+// data types used to index the routers and links within the noc
 
 struct noc_router_id_tag;
 struct noc_link_id_tag;

vpr/src/base/vpr_api.cpp

@@ -503,17 +503,18 @@ void vpr_setup_clock_networks(t_vpr_setup& vpr_setup, const t_arch& Arch) {
  *             description file.
  */
 void vpr_setup_noc(const t_vpr_setup& vpr_setup, const t_arch& arch) {
-    t_draw_state* draw_state = get_draw_state_vars();
-
     // check if the user provided the option to model the noc
     if (vpr_setup.NocOpts.noc == true) {
         // create the NoC model based on the user description from the arch file
         setup_noc(arch);
 
+#ifndef NO_GRAPHICS
         // setup the graphics
         // if the user turned on "noc" in the command line, then we also want them to have the option to display the noc, so set that option here to be able to display it.
         // if the "noc" was not turned on, then we don't need to provide the user with the option to display it
+        t_draw_state* draw_state = get_draw_state_vars();
         draw_state->show_noc_button = true;
+#endif
     }
 }
 

vpr/src/draw/draw_noc.cpp

@@ -1,9 +1,11 @@
 
-#include "draw_noc.h"
-#include "globals.h"
-#include "noc_storage.h"
-#include "vpr_error.h"
-#include "vtr_math.h"
+#ifndef NO_GRAPHICS
+
+#    include "draw_noc.h"
+#    include "globals.h"
+#    include "noc_storage.h"
+#    include "vpr_error.h"
+#    include "vtr_math.h"
 
 /**
  * @brief For a NoC that is undirected, each connection between routers has two
@@ -226,3 +228,5 @@ void shift_noc_link(noc_link_draw_coords& link_coords, NocLinkShift link_shift_d
 
     return;
 }
+
+#endif

vpr/src/draw/draw_noc.h

@@ -9,7 +9,9 @@
 #include <iostream>
 #include <vector>
 
-#include "draw.h"
+#ifndef NO_GRAPHICS
+
+#    include "draw.h"
 
 // defines the length of a reference horizontal line that is used in a cross product to calculate the angle between this line and a noc link to be drawn
 const double HORIZONTAL_LINE_LENGTH(5.0);
@@ -51,4 +53,6 @@ NocLinkType determine_noc_link_type(ezgl::point2d link_start_point, ezgl::point2
 
 void shift_noc_link(noc_link_draw_coords& link_coords, NocLinkShift link_shift_direction, NocLinkType link_type, double noc_connection_marker_quarter_width, double noc_connection_marker_quarter_height);
 
+#endif
+
 #endif

vpr/test/test_setup_noc.cpp

@@ -10,7 +10,6 @@
 
 /*Re-defining static functions in setup_noc.cpp that are tested here*/
 
-
 static void identify_and_store_noc_router_tile_positions(const DeviceGrid& device_grid, std::vector<t_noc_router_tile_position>& list_of_noc_router_tiles, std::string noc_router_tile_name) {
     int grid_width = device_grid.width();
     int grid_height = device_grid.height();
@@ -52,8 +51,6 @@ static void identify_and_store_noc_router_tile_positions(const DeviceGrid& devic
     return;
 }
 
-
-
 static void create_noc_routers(const t_noc_inf& noc_info, NocStorage* noc_model, std::vector<t_noc_router_tile_position>& list_of_noc_router_tiles) {
     // keep track of the shortest distance between a logical router and the curren physical router tile
     // also keep track of the corresponding physical router tile index (within the list)