From 864cc54aa43d40368483e58cb7e04a42c96f9ded Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Henrik=20Rydg=C3=A5rd?= <hrydgard@gmail.com>
Date: Mon, 9 Nov 2020 09:02:59 +0100
Subject: [PATCH] Remove ForceGL2 flag. Add comments... hm.

---
 Common/GPU/OpenGL/GLFeatures.cpp | 9 ---------
 Common/GPU/OpenGL/GLFeatures.h   | 1 -
 Common/GPU/OpenGL/thin3d_gl.cpp  | 4 +++-
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/Common/GPU/OpenGL/GLFeatures.cpp b/Common/GPU/OpenGL/GLFeatures.cpp
index df8632779c..9b7070e458 100644
--- a/Common/GPU/OpenGL/GLFeatures.cpp
+++ b/Common/GPU/OpenGL/GLFeatures.cpp
@@ -541,15 +541,6 @@ void CheckGLExtensions() {
 		}
 	}
 
-#ifdef __APPLE__
-	if (!gl_extensions.IsGLES && !gl_extensions.IsCoreContext) {
-		// Apple doesn't allow OpenGL 3.x+ in compatibility contexts.
-		// TODO: But are we still ever creating these on Apple? Would really
-		// like to kill this ForceGL2 flag.
-		gl_extensions.ForceGL2 = true;
-	}
-#endif
-
 	ProcessGPUFeatures();
 
 	int error = glGetError();
diff --git a/Common/GPU/OpenGL/GLFeatures.h b/Common/GPU/OpenGL/GLFeatures.h
index f4da0ce221..996285073d 100644
--- a/Common/GPU/OpenGL/GLFeatures.h
+++ b/Common/GPU/OpenGL/GLFeatures.h
@@ -38,7 +38,6 @@ struct GLExtensions {
 	bool IsGLES;
 	bool IsCoreContext;
 	bool GLES3;  // true if the full OpenGL ES 3.0 is supported
-	bool ForceGL2;
 
 	// OES
 	bool OES_depth24;
diff --git a/Common/GPU/OpenGL/thin3d_gl.cpp b/Common/GPU/OpenGL/thin3d_gl.cpp
index e789c987df..b0da1da636 100644
--- a/Common/GPU/OpenGL/thin3d_gl.cpp
+++ b/Common/GPU/OpenGL/thin3d_gl.cpp
@@ -632,7 +632,7 @@ OpenGLContext::OpenGLContext() {
 			}
 		}
 	} else {
-		if (!gl_extensions.ForceGL2 || gl_extensions.IsCoreContext) {
+		if (gl_extensions.IsCoreContext) {
 			if (gl_extensions.VersionGEThan(3, 3, 0)) {
 				shaderLanguageDesc_.shaderLanguage = ShaderLanguage::GLSL_3xx;
 				shaderLanguageDesc_.glslVersionNumber = 330;
@@ -645,12 +645,14 @@ OpenGLContext::OpenGLContext() {
 				shaderLanguageDesc_.varying_fs = "in";
 				shaderLanguageDesc_.attribute = "in";
 			} else if (gl_extensions.VersionGEThan(3, 0, 0)) {
+				// Hm, I think this is wrong. This should be outside "if (gl_extensions.IsCoreContext)".
 				shaderLanguageDesc_.shaderLanguage = ShaderLanguage::GLSL_1xx;
 				shaderLanguageDesc_.glslVersionNumber = 130;
 				shaderLanguageDesc_.fragColor0 = "fragColor0";
 				shaderLanguageDesc_.bitwiseOps = true;
 				shaderLanguageDesc_.texelFetch = "texelFetch";
 			} else {
+				// This too...
 				shaderLanguageDesc_.shaderLanguage = ShaderLanguage::GLSL_1xx;
 				shaderLanguageDesc_.glslVersionNumber = 110;
 				if (gl_extensions.EXT_gpu_shader4) {