-
Notifications
You must be signed in to change notification settings - Fork 403
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gpu: More pre-work to combine Printf and GPUAV #8611
gpu: More pre-work to combine Printf and GPUAV #8611
Conversation
CI Vulkan-ValidationLayers build queued with queue ID 266267. |
@@ -526,3 +526,165 @@ void CommandBuffer::PostProcess(VkQueue queue, const Location &loc) { | |||
} | |||
|
|||
} // namespace gpuav | |||
|
|||
|
|||
namespace gpu_tracker { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the final move will be to remove this namespace
but keep the logic in this file (where it belongs)
CI Vulkan-ValidationLayers build # 17621 running. |
@@ -1362,7 +1369,7 @@ bool GpuShaderInstrumentor::InstrumentShader(const vvl::span<const uint32_t> &in | |||
gpu::spirv::Module module(input_spirv, debug_report, module_settings); | |||
|
|||
// For now, we don't yet support (or have tested) combining GPU-AV and DebugPrintf, so have 2 paths here | |||
const bool is_debug_printf = container_type == LayerObjectTypeDebugPrintf; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could just use debug_printf_enabled
directly instead of is_debug_printf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya, we are going to need to change this from is_debug_printf
to is_only_debug_printf
anyways.. basically my main goal was to just remove anyone trying to read from LayerObjectTypeDebugPrintf
lol
CI Vulkan-ValidationLayers build # 17621 passed. |
Takes work from and brings out things that can be done now that will make the "final merge" PR more readable
cedb681