From 7d75a7310507fb8e21d65696306dfafbd6021078 Mon Sep 17 00:00:00 2001
From: Gregory Hainaut <gregory.hainaut@gmail.com>
Date: Sun, 21 May 2017 19:23:05 +0200
Subject: [PATCH] cmake: blacklist GCC 7.0 and GCC 7.1 (based on a small
 testcase)

GCC bug => https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80799

Close #1937
---
 cmake/ApiValidation.cmake   | 101 ++++++++++++++++++++++++++++++++++++
 cmake/SearchForStuff.cmake  |   6 +++
 plugins/GSdx/CMakeLists.txt |   4 --
 3 files changed, 107 insertions(+), 4 deletions(-)

diff --git a/cmake/ApiValidation.cmake b/cmake/ApiValidation.cmake
index 489e1413dd..e7a4f99a77 100644
--- a/cmake/ApiValidation.cmake
+++ b/cmake/ApiValidation.cmake
@@ -6,6 +6,79 @@ set(wx_sdl_c_code "
 #endif
 ")
 
+set(gcc7_mmx_code "
+#include <stdio.h>
+#include <stdint.h>
+#include <xmmintrin.h>
+#include <emmintrin.h>
+
+class alignas(16) GSVector4i
+{
+public:
+    __m128i m;
+
+	explicit GSVector4i(__m128i m)
+	{
+		this->m = m;
+	}
+
+	static void storel(void* p, const GSVector4i& v)
+	{
+		_mm_storel_epi64((__m128i*)p, v.m);
+	}
+
+	static GSVector4i loadl(const void* p)
+	{
+		return GSVector4i(_mm_loadl_epi64((__m128i*)p));
+	}
+
+	bool eq(const GSVector4i& v) const
+	{
+		return _mm_movemask_epi8(_mm_cmpeq_epi32(m, v.m)) == 0xffff;
+	}
+};
+
+
+union GIFRegTRXPOS
+{
+	unsigned long long u64;
+	void operator = (const GSVector4i& v) {GSVector4i::storel(this, v);}
+	bool operator != (const union GIFRegTRXPOS& r) const {return !((GSVector4i)r).eq(*this);}
+	operator GSVector4i() const {return GSVector4i::loadl(this);}
+};
+
+extern GIFRegTRXPOS	TRXPOS;
+GIFRegTRXPOS TRXPOS = {};
+
+void GIFRegHandlerTRXPOS(const GIFRegTRXPOS&  p)
+{
+	if(p != TRXPOS)
+	{
+        printf(\"foo\");
+	}
+
+	TRXPOS = (GSVector4i)p;
+}
+
+int main()
+{
+    GIFRegTRXPOS r = {};
+    GIFRegHandlerTRXPOS(r);
+
+    uint16_t fpu[16] = {0};
+    __asm__ __volatile__(\"fstenv %0\" : \"=m\"(fpu));
+
+    bool ok = fpu[4] == 0xFFFF;
+
+    if (!ok) {
+        printf(\"Wrong MMX state !\");
+        exit(1);
+    }
+
+    return 0;
+}
+")
+
 function(WX_vs_SDL)
     file(WRITE "${CMAKE_BINARY_DIR}/wx_sdl.c" "${wx_sdl_c_code}")
     enable_language(C)
@@ -25,3 +98,31 @@ function(WX_vs_SDL)
         Please use -DSDL2_API=FALSE")
     endif()
 endfunction()
+
+function(GCC7_BUG)
+    # try_run doesn't work when cross-compiling is enabled. It is completely silly in our case
+    # as i386 binaries are 100% fine on x64.
+    set(OLD_CMAKE_CROSSCOMPILING ${CMAKE_CROSSCOMPILING})
+    set(CMAKE_CROSSCOMPILING 0)
+
+    set(IN "${CMAKE_BINARY_DIR}/gcc7_mmx.cpp")
+    file(WRITE "${IN}" "${gcc7_mmx_code}")
+
+    enable_language(CXX)
+
+    try_run(
+            run_result
+            compile_result_unused
+            "${CMAKE_BINARY_DIR}"
+            "${IN}"
+            CMAKE_FLAGS "-DCOMPILE_DEFINITIONS:STRING=-msse -msse2 -O2 -m32 -march=i686"
+        )
+
+    if (${run_result})
+        message(FATAL_ERROR "GCC 7.0/7.1  generates invalid code => https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80799\n"
+            "You can either backport the fix or swith to another version of GCC.")
+    endif()
+
+    set(CMAKE_CROSSCOMPILING ${OLD_CMAKE_CROSSCOMPILING})
+
+endfunction()
diff --git a/cmake/SearchForStuff.cmake b/cmake/SearchForStuff.cmake
index 78a4af480e..3bf2f2182d 100644
--- a/cmake/SearchForStuff.cmake
+++ b/cmake/SearchForStuff.cmake
@@ -191,4 +191,10 @@ include_directories(${CMAKE_SOURCE_DIR}/common/include
 # Note: wxWidgets_INCLUDE_DIRS must be defined
 #----------------------------------------
 include(ApiValidation)
+
 WX_vs_SDL()
+
+# Blacklist bad GCC
+if(GCC_VERSION VERSION_EQUAL "7.0" OR GCC_VERSION VERSION_EQUAL "7.1")
+    GCC7_BUG()
+endif()
diff --git a/plugins/GSdx/CMakeLists.txt b/plugins/GSdx/CMakeLists.txt
index 9b7b0645f9..34291f26a2 100644
--- a/plugins/GSdx/CMakeLists.txt
+++ b/plugins/GSdx/CMakeLists.txt
@@ -27,10 +27,6 @@ if(CMAKE_COMPILER_IS_GNUCXX)
 	if (${GCC_VERSION} VERSION_LESS "5.1")
 		set(GSdxFinalFlags ${GSdxFinalFlags} -fabi-version=6)
 	endif()
-	# github issue 1937
-	if (${GCC_VERSION} VERSION_GREATER "7.0")
-		set(GSdxFinalFlags ${GSdxFinalFlags} -fno-gcse-lm -fno-tree-pre)
-	endif()
 endif()
 
 if(XDG_STD)