From 39456bceba97763d79970093fe2e5a4f0acb153e Mon Sep 17 00:00:00 2001 From: baldurk Date: Tue, 16 Aug 2022 15:34:27 +0100 Subject: [PATCH] Bake in Vulkan_Debug_StepToDebugValue * The behaviour with this set to true is significantly better for lightly- optimised code as comes out of HLSL legalisation, with little penalty for other code. --- renderdoc/driver/shaders/spirv/spirv_debug.cpp | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/renderdoc/driver/shaders/spirv/spirv_debug.cpp b/renderdoc/driver/shaders/spirv/spirv_debug.cpp index 6605388be..0eb458a0c 100644 --- a/renderdoc/driver/shaders/spirv/spirv_debug.cpp +++ b/renderdoc/driver/shaders/spirv/spirv_debug.cpp @@ -32,9 +32,6 @@ #include "spirv_op_helpers.h" #include "var_dispatch_helpers.h" -RDOC_CONFIG(bool, Vulkan_Debug_StepToDebugValue, false, - "Treat DebugValue as a steppable executable instruction."); - static bool ContainsNaNInf(const ShaderVariable &var) { bool ret = false; @@ -588,15 +585,7 @@ void ThreadState::SkipIgnoredInstructions() { if(debugger.IsDebugExtInstSet(Id::fromWord(it.word(3)))) { - if(Vulkan_Debug_StepToDebugValue()) - { - if(ShaderDbg(it.word(4)) != ShaderDbg::Value || !debugger.InDebugScope(nextInstruction)) - { - nextInstruction++; - continue; - } - } - else + if(ShaderDbg(it.word(4)) != ShaderDbg::Value || !debugger.InDebugScope(nextInstruction)) { nextInstruction++; continue;