From 4e204a9a1a6bc1f0ef5402f3163d3d27338395c2 Mon Sep 17 00:00:00 2001 From: Robin Kertels Date: Sat, 24 Sep 2022 21:14:34 +0200 Subject: [PATCH] VideoCommon: Remove special treatment for Android in video settings --- .../dolphinemu/features/settings/model/BooleanSetting.java | 2 +- Source/Android/app/src/main/res/values/strings.xml | 4 ++-- Source/Core/Core/Config/GraphicsSettings.cpp | 7 ------- Source/Core/VideoCommon/VideoConfig.h | 4 ---- 4 files changed, 3 insertions(+), 14 deletions(-) diff --git a/Source/Android/app/src/main/java/org/dolphinemu/dolphinemu/features/settings/model/BooleanSetting.java b/Source/Android/app/src/main/java/org/dolphinemu/dolphinemu/features/settings/model/BooleanSetting.java index 0c02f1b9a0..64f380988d 100644 --- a/Source/Android/app/src/main/java/org/dolphinemu/dolphinemu/features/settings/model/BooleanSetting.java +++ b/Source/Android/app/src/main/java/org/dolphinemu/dolphinemu/features/settings/model/BooleanSetting.java @@ -197,7 +197,7 @@ public enum BooleanSetting implements AbstractBooleanSetting GFX_ENABLE_VALIDATION_LAYER(Settings.FILE_GFX, Settings.SECTION_GFX_SETTINGS, "EnableValidationLayer", false), GFX_BACKEND_MULTITHREADING(Settings.FILE_GFX, Settings.SECTION_GFX_SETTINGS, - "BackendMultithreading", false), + "BackendMultithreading", true), GFX_WAIT_FOR_SHADERS_BEFORE_STARTING(Settings.FILE_GFX, Settings.SECTION_GFX_SETTINGS, "WaitForShadersBeforeStarting", false), GFX_SAVE_TEXTURE_CACHE_TO_STATE(Settings.FILE_GFX, Settings.SECTION_GFX_SETTINGS, diff --git a/Source/Android/app/src/main/res/values/strings.xml b/Source/Android/app/src/main/res/values/strings.xml index c54a2ead03..23eccd0c10 100644 --- a/Source/Android/app/src/main/res/values/strings.xml +++ b/Source/Android/app/src/main/res/values/strings.xml @@ -339,8 +339,8 @@ Crop Crops the picture from its native aspect ratio to 4:3 or 16:9. If unsure, leave this unchecked. Enable Progressive Scan - Backend Multithreading - Enables graphics backend multithreading (Vulkan only). May affect performance. If unsure, leave this unchecked. + Backend Multithreading + Enables graphics backend multithreading (Vulkan only). May affect performance. If unsure, leave this checked. Defer EFB Cache Invalidation Defers invalidation of the EFB access cache until a GPU synchronization command is executed. May improve performance in some games at the cost of stability. If unsure, leave this unchecked. Manual Texture Sampling diff --git a/Source/Core/Core/Config/GraphicsSettings.cpp b/Source/Core/Core/Config/GraphicsSettings.cpp index 4bbcf99413..866719a0ac 100644 --- a/Source/Core/Core/Config/GraphicsSettings.cpp +++ b/Source/Core/Core/Config/GraphicsSettings.cpp @@ -69,17 +69,10 @@ const Info GFX_BORDERLESS_FULLSCREEN{{System::GFX, "Settings", "Borderless const Info GFX_ENABLE_VALIDATION_LAYER{{System::GFX, "Settings", "EnableValidationLayer"}, false}; -#if defined(ANDROID) -const Info GFX_BACKEND_MULTITHREADING{{System::GFX, "Settings", "BackendMultithreading"}, - false}; -const Info GFX_COMMAND_BUFFER_EXECUTE_INTERVAL{ - {System::GFX, "Settings", "CommandBufferExecuteInterval"}, 0}; -#else const Info GFX_BACKEND_MULTITHREADING{{System::GFX, "Settings", "BackendMultithreading"}, true}; const Info GFX_COMMAND_BUFFER_EXECUTE_INTERVAL{ {System::GFX, "Settings", "CommandBufferExecuteInterval"}, 100}; -#endif const Info GFX_SHADER_CACHE{{System::GFX, "Settings", "ShaderCache"}, true}; const Info GFX_WAIT_FOR_SHADERS_BEFORE_STARTING{ diff --git a/Source/Core/VideoCommon/VideoConfig.h b/Source/Core/VideoCommon/VideoConfig.h index 105840bafa..e76e87af9b 100644 --- a/Source/Core/VideoCommon/VideoConfig.h +++ b/Source/Core/VideoCommon/VideoConfig.h @@ -159,11 +159,7 @@ struct VideoConfig final bool bEnableValidationLayer = false; // Multithreaded submission, currently only supported with Vulkan. -#if defined(ANDROID) - bool bBackendMultithreading = false; -#else bool bBackendMultithreading = true; -#endif // Early command buffer execution interval in number of draws. // Currently only supported with Vulkan.