add missing stuff for MSVC

- tests-main library changed to STATIC
 - MSVC dlls would require to do extra import/export effort

Author: Sylvain Garcia

cmake/FindCoroutines.cmake

@@ -115,6 +115,9 @@ check_cxx_compiler_flag(-fcoroutines _CXX_COROUTINES_SUPPORTS_CORO_FLAG)
 
 if(_CXX_COROUTINES_SUPPORTS_MS_FLAG)
     set(_CXX_COROUTINES_EXTRA_FLAGS "/await")
+    if(CMAKE_CL_64)
+        list(APPEND _CXX_COROUTINES_EXTRA_FLAGS "/await:heapelide")
+    endif()
 elseif(_CXX_COROUTINES_SUPPORTS_TS_FLAG)
     set(_CXX_COROUTINES_EXTRA_FLAGS "-fcoroutines-ts")
 elseif(_CXX_COROUTINES_SUPPORTS_CORO_FLAG)

lib/CMakeLists.txt

@@ -144,6 +144,14 @@ if(WIN32)
         socket_recv_from_operation.cpp
     )
     list(APPEND sources ${win32Sources})
+
+    list(APPEND libraries Ws2_32 Mswsock Synchronization)
+    list(APPEND compile_options /EHsc)
+
+    if("${MSVC_VERSION}" VERSION_GREATER_EQUAL 1900)
+      # TODO remove this when experimental/non-experimental include are fixed
+      list(APPEND compile_definition _SILENCE_EXPERIMENTAL_FILESYSTEM_DEPRECATION_WARNING=1)
+    endif()
 endif()
 
 add_library(cppcoro
@@ -158,9 +166,11 @@ target_include_directories(cppcoro PUBLIC
 	$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
 	$<INSTALL_INTERFACE:include>)
 target_compile_features(cppcoro PUBLIC cxx_std_20)
+target_compile_definitions(cppcoro PUBLIC ${compile_definition})
+target_compile_options(cppcoro PUBLIC ${compile_options})
 
 find_package(Coroutines COMPONENTS Experimental Final REQUIRED)
-target_link_libraries(cppcoro PUBLIC std::coroutines)
+target_link_libraries(cppcoro PUBLIC std::coroutines ${libraries})
 
 install(TARGETS cppcoro EXPORT cppcoroTargets
 	LIBRARY DESTINATION lib

test/CMakeLists.txt

@@ -5,7 +5,7 @@ include(${CMAKE_CURRENT_LIST_DIR}/doctest/doctest.cmake)
 
 find_package(Threads REQUIRED)
 
-add_library(tests-main SHARED
+add_library(tests-main STATIC
 	main.cpp
 	counted.cpp
 )