cpp/openttd-patchpack/source Changeset - r25075:20911544ed27

Changeset - r25075:20911544ed27

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

Milek7 - 3 years ago 2021-03-19 23:08:03
Milek7@users.noreply.github.com

Change: Do not disallow persistent buffer mapping on AMD GPUs, as it is actually faster.

1 file changed with 0 insertions and 10 deletions:

src/video/opengl.cpp

0 comments (0 inline, 0 general)

src/video/opengl.cpp

➞

Show inline comments

@@ @@ -13,8 +13,6 @@ @@
 /* Define to disable buffer syncing. Will increase max fast forward FPS but produces artifacts. Mainly useful for performance testing. */
 // #define NO_GL_BUFFER_SYNC
 /* Define to enable persistent buffer mapping on AMD GPUs. */
 // #define GL_MAP_PERSISTENT_AMD
 /* Define to allow software rendering backends. */
 // #define GL_ALLOW_SOFTWARE_RENDERER
@@ @@ -577,14 +575,6 @@ const char *OpenGLBackend::Init() @@
 	this->persistent_mapping_supported = this->persistent_mapping_supported && (IsOpenGLVersionAtLeast(3, 2) || IsOpenGLExtensionSupported("GL_ARB_sync"));
 #endif
 #ifndef GL_MAP_PERSISTENT_AMD
 	if (this->persistent_mapping_supported && (strstr(vend, "AMD") != nullptr || strstr(renderer, "Radeon") != nullptr)) {
 		/* AMD GPUs seem to perform badly with persistent buffer mapping, disable it for them. */
 		DEBUG(driver, 3, "OpenGL: Detected AMD GPU, not using persistent buffer mapping due to performance problems");
 		this->persistent_mapping_supported = false;
+	}
 #endif
 	if (this->persistent_mapping_supported && !BindPersistentBufferExtensions()) {
 		DEBUG(driver, 1, "OpenGL claims to support persistent buffer mapping but doesn't export all functions, not using persistent mapping.");
 		this->persistent_mapping_supported = false;

0 comments (0 inline, 0 general)