fixed VS 2017 workarounds

Author: Tobias Loew

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.13)
+cmake_minimum_required(VERSION 3.12)
 project(cppcoro LANGUAGES CXX)
 
 set(CMAKE_CXX_EXTENSIONS OFF)

include/cppcoro/detail/when_all_task.hpp

@@ -77,7 +77,7 @@ namespace cppcoro
 				assert(false);
 			}
 
-#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'00'00000
+#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'20'00000
 			// HACK: This is needed to work around a bug in MSVC 2017.7/2017.8.
 			// See comment in make_when_all_task below.
 			template<typename Awaitable>
@@ -302,7 +302,7 @@ namespace cppcoro
 			std::enable_if_t<!std::is_void_v<RESULT>, int> = 0>
 		when_all_task<RESULT> make_when_all_task(AWAITABLE awaitable)
 		{
-#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'00'00000
+#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'20'00000
 			// HACK: Workaround another bug in MSVC where the expression 'co_yield co_await x' seems
 			// to completely ignore the co_yield an never calls promise.yield_value().
 			// The coroutine seems to be resuming the 'co_await' after the 'co_yield'
@@ -330,7 +330,7 @@ namespace cppcoro
 			std::enable_if_t<!std::is_void_v<RESULT>, int> = 0>
 		when_all_task<RESULT> make_when_all_task(std::reference_wrapper<AWAITABLE> awaitable)
 		{
-#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'00'00000
+#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'20'00000
 			// HACK: Workaround another bug in MSVC where the expression 'co_yield co_await x' seems
 			// to completely ignore the co_yield and never calls promise.yield_value().
 			// The coroutine seems to be resuming the 'co_await' after the 'co_yield'

include/cppcoro/sync_wait.hpp

@@ -18,7 +18,7 @@ namespace cppcoro
 	auto sync_wait(AWAITABLE&& awaitable)
 		-> typename cppcoro::awaitable_traits<AWAITABLE&&>::await_result_t
 	{
-#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'00'00000
+#if CPPCORO_COMPILER_MSVC && CPPCORO_COMPILER_MSVC < 19'20'00000
 		// HACK: Need to explicitly specify template argument to make_sync_wait_task
 		// here to work around a bug in MSVC when passing parameters by universal
 		// reference to a coroutine which causes the compiler to think it needs to