Remove hack workaround for co_return <void-expr> under clang.

lewissbaker · lewissbaker · commit 7513d9cf0c5f · 2017-08-02T21:30:17.000+09:30
This bug has been fixed in clang-6.0.
diff --git a/include/cppcoro/shared_lazy_task.hpp b/include/cppcoro/shared_lazy_task.hpp
@@ -513,7 +513,7 @@ namespace cppcoro
 		co_return co_await std::move(t);
 	}
 
-#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019 || CPPCORO_COMPILER_CLANG
+#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019
 	// HACK: Workaround for broken MSVC that doesn't execute <expr> in 'co_return <expr>;'.
 	inline shared_lazy_task<void> make_shared_task(lazy_task<void> t)
 	{
diff --git a/include/cppcoro/shared_task.hpp b/include/cppcoro/shared_task.hpp
@@ -476,7 +476,7 @@ namespace cppcoro
 		co_return co_await std::move(t);
 	}
 
-#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019 || CPPCORO_COMPILER_CLANG
+#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019
 	// HACK: Work around bug in MSVC handling of 'co_return <expr>' for void-returning expression.
 	// It doesn't actually evaluate <expr> before calling <promise>.return_void().
 	inline shared_task<void> make_shared_task(task<void> t)

Original file line number	Diff line number	Diff line change
`@@ -513,7 +513,7 @@ namespace cppcoro`
`513`	`513`	`co_return co_await std::move(t);`
`514`	`514`	`}`
`515`	`515`
`516`		`-#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019 \|\| CPPCORO_COMPILER_CLANG`
	`516`	`+#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019`
`517`	`517`	`// HACK: Workaround for broken MSVC that doesn't execute <expr> in 'co_return <expr>;'.`
`518`	`518`	`inline shared_lazy_task<void> make_shared_task(lazy_task<void> t)`
`519`	`519`	`{`
Original file line number	Diff line number	Diff line change
`@@ -476,7 +476,7 @@ namespace cppcoro`
`476`	`476`	`co_return co_await std::move(t);`
`477`	`477`	`}`
`478`	`478`
`479`		`-#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019 \|\| CPPCORO_COMPILER_CLANG`
	`479`	`+#if defined(_MSC_VER) && _MSC_FULL_VER <= 191025019`
`480`	`480`	`// HACK: Work around bug in MSVC handling of 'co_return <expr>' for void-returning expression.`
`481`	`481`	`// It doesn't actually evaluate <expr> before calling <promise>.return_void().`
`482`	`482`	`inline shared_task<void> make_shared_task(task<void> t)`