Merge pull request lnis-uofu#2446 from oscarcheng105/oscar_windows_support

[WIP] Minimum build for Windows support

Author: tangxifan

.gitignore

@@ -54,3 +54,7 @@ vpr/vpr
 node_modules
 package-lock.json
 /_*/ 
+.vs/
+openfpga_flow/.vs/
+vcpkg_installed/
+CMakeSettings.json

CMakeLists.txt

@@ -44,6 +44,17 @@ set_property(CACHE VTR_ASSERT_LEVEL PROPERTY STRINGS 0 1 2 3 4)
 #Create the project 
 project("OpenFPGA-tool-suites" C CXX)
 
+# Windows fixes
+if (MSVC)
+    add_compile_definitions(
+        NOMINMAX
+        _CRT_SECURE_NO_WARNINGS
+    )
+else()
+    add_compile_definitions(NOMINMAX)
+endif()
+
+
 # Version number
 file (STRINGS "VERSION.md" VERSION_NUMBER)
 string (REPLACE "." ";" VERSION_LIST ${VERSION_NUMBER})
@@ -81,7 +92,12 @@ message(STATUS "CMAKE_OPENFPGA_WITH_INSTALLER: ${OPENFPGA_WITH_INSTALLER}")
 message(STATUS "CMAKE_OPENFPGA_INSTALL_DOC: ${OPENFPGA_INSTALL_DOC}")
 
 # Options pass on to VTR
+
+if(MSVC)
+set(WITH_ABC OFF CACHE BOOL "Enable building ABC in Verilog-to-Routing")
+else()
 set(WITH_ABC ON CACHE BOOL "Enable building ABC in Verilog-to-Routing")
+endif()
 set(WITH_ODIN OFF CACHE BOOL "Enable building Odin in Verilog-to-Routing")
 set(ODIN_DEBUG OFF CACHE BOOL "Enable building odin with debug flags in Verilog-to-Routing")
 set(ODIN_WARN OFF CACHE BOOL "Enable building odin with extra warning flags in Verilog-to-Routing")

cmake/modules/SwigLib.cmake

@@ -84,10 +84,12 @@ function(SwigLib)
   get_target_property(GEN_SRCS ${ARG_NAME} SOURCES)
 
   foreach(GEN_SRC ${GEN_SRCS})
-    set_source_files_properties(${GEN_SRC}
-      PROPERTIES
-        COMPILE_OPTIONS "-Wno-cast-qual;-Wno-missing-braces"
-    )
+    if(NOT MSVC)
+        set_source_files_properties(${GEN_SRC}
+          PROPERTIES
+            COMPILE_OPTIONS "-Wno-cast-qual;-Wno-missing-braces"
+        )
+    endif()
   endforeach()
 
   # These includes are always needed.

libs/libini/CMakeLists.txt

@@ -16,7 +16,7 @@ endif()
 target_include_directories(libini INTERFACE ${LIB_INCLUDE_DIRS})
 set_target_properties(libini PROPERTIES PREFIX "")
 
-install(TARGETS libini
+install(DIRECTORY src/
         DESTINATION bin
         COMPONENT openfpga_package
-)
+)

libs/libopenfpgashell/CMakeLists.txt

@@ -3,7 +3,11 @@ cmake_minimum_required(VERSION 3.9)
 project("libopenfpgashell")
 
 # We need readline to compile
-find_package(Readline REQUIRED)
+if(WIN32)
+  find_package(unofficial-readline-win32 CONFIG REQUIRED)
+else()
+  find_package(Readline REQUIRED)
+endif()
 
 file(GLOB_RECURSE EXEC_SOURCES test/*.cpp)
 file(GLOB_RECURSE LIB_SOURCES src/*/*.cpp)
@@ -20,11 +24,18 @@ add_library(libopenfpgashell STATIC
 target_include_directories(libopenfpgashell PUBLIC ${LIB_INCLUDE_DIRS})
 set_target_properties(libopenfpgashell PROPERTIES PREFIX "") #Avoid extra 'lib' prefix
 
-#Specify link-time dependancies
+# Specify link-time dependencies
 target_link_libraries(libopenfpgashell
                       libopenfpgautil
-                      libvtrutil
-                      readline)
+                      libvtrutil)
+
+if(WIN32)
+  target_link_libraries(libopenfpgashell
+                        unofficial::readline-win32::readline)
+else()
+  target_link_libraries(libopenfpgashell
+                        readline)
+endif()
 
 # Create the test executable
 foreach(testsourcefile ${EXEC_SOURCES})

libs/libopenfpgautil/src/openfpga_digest.cpp

@@ -2,6 +2,10 @@
  * This file includes functions that handles the file outputting
  * in OpenFPGA framework
  *******************************************************************/
+#ifdef _WIN32
+#include <direct.h>  // Windows: provides mkdir() (POSIX uses <sys/stat.h>)
+#endif
+
 #include <sys/stat.h>
 
 #include <algorithm>

libs/libpcf/src/base/pcf_custom_command.cpp

@@ -172,7 +172,8 @@ vtr::Point<int> PcfCustomCommand::custom_decimal_mode_segments_range_to_int(
   openfpga::PortParser port_parser(
     range, openfpga::PORT_PARSER_SUPPORT_ALL_FORMAT, true);
   openfpga::BasicPort pin = port_parser.port();
-  vtr::Point<int> range_int = {pin.get_lsb(), pin.get_msb()};
+  vtr::Point<int> range_int = {static_cast<int>(pin.get_lsb()),
+                               static_cast<int>(pin.get_msb())};
   return range_int;
 }
 bool PcfCustomCommand::custom_decimal_mode_segments_valid(

openfpga/src/base/openfpga_basic.cpp

@@ -89,13 +89,22 @@ int call_external_command(const Command& cmd,
   // Refer
   // https://pubs.opengroup.org/onlinepubs/009695399/functions/waitpid.html
   int status = system(cmd_ss.c_str());
+
+#ifdef _WIN32
+  // On Windows, system() returns the command exit code directly.
+  // Becareful if the final status is 2 or beyond, program will not error
+  // as it is treated as CMD_EXEC_MINOR_ERROR
+  return status;
+#else
   if (WIFEXITED(status)) {
     // This is normal program exit, WEXITSTATUS() will help you shift the status
     // accordingly (status >> 8)
     // Becareful if the final status is 2 or beyond, program will not error
     // as it is treated as CMD_EXEC_MINOR_ERROR
     return WEXITSTATUS(status);
   }
+#endif
+
   // Program maybe terminated because of various killed or stopped signal
   return CMD_EXEC_FATAL_ERROR;
 }

openfpga/src/base/openfpga_bitstream_command_template.h

@@ -1,5 +1,8 @@
 #ifndef OPENFPGA_BITSTREAM_COMMAND_TEMPLATE_H
 #define OPENFPGA_BITSTREAM_COMMAND_TEMPLATE_H
+
+#include "openfpga_windows_compatibility.h"
+
 /********************************************************************
  * Add commands to the OpenFPGA shell interface,
  * in purpose of generate Verilog netlists modeling the full FPGA fabric

openfpga/src/base/openfpga_sdc_command_template.h

@@ -1,5 +1,8 @@
 #ifndef OPENFPGA_SDC_COMMAND_TEMPLATE_H
 #define OPENFPGA_SDC_COMMAND_TEMPLATE_H
+
+#include "openfpga_windows_compatibility.h"
+
 /********************************************************************
  * Add commands to the OpenFPGA shell interface,
  * in purpose of generate SDC files