From bf3db5a2c92731e00393df179293b4e04e74eedd Mon Sep 17 00:00:00 2001 From: ssjia Date: Thu, 6 Nov 2025 17:06:42 -0800 Subject: [PATCH] [ET-VK][ez] Fix ETVKNODEBUG lint signal Title says it all! Differential Revision: [D86477050](https://our.internmc.facebook.com/intern/diff/D86477050/) [ghstack-poisoned] --- .../runtime/graph/ops/glsl/reduce_per_row_buffer.glsl | 2 -- .../runtime/graph/ops/glsl/sdpa_kv_cache_update.glsl | 4 ---- .../test/custom_ops/glsl/packed_int32_canvas_buffer.glsl | 8 -------- .../custom_ops/glsl/packed_int32_canvas_texture3d.glsl | 8 -------- 4 files changed, 22 deletions(-) diff --git a/backends/vulkan/runtime/graph/ops/glsl/reduce_per_row_buffer.glsl b/backends/vulkan/runtime/graph/ops/glsl/reduce_per_row_buffer.glsl index 5d377cf1f1a..af5f5f661e7 100644 --- a/backends/vulkan/runtime/graph/ops/glsl/reduce_per_row_buffer.glsl +++ b/backends/vulkan/runtime/graph/ops/glsl/reduce_per_row_buffer.glsl @@ -57,8 +57,6 @@ $else: $if OUTPUT_IS_INDICES: #define OUTPUT_IS_INDICES -#extension GL_EXT_debug_printf : require - void main() { const uint out_bufi = gl_GlobalInvocationID.y; diff --git a/backends/vulkan/runtime/graph/ops/glsl/sdpa_kv_cache_update.glsl b/backends/vulkan/runtime/graph/ops/glsl/sdpa_kv_cache_update.glsl index b780cdce6fe..5f7e4c2719d 100644 --- a/backends/vulkan/runtime/graph/ops/glsl/sdpa_kv_cache_update.glsl +++ b/backends/vulkan/runtime/graph/ops/glsl/sdpa_kv_cache_update.glsl @@ -14,10 +14,6 @@ ${define_required_extensions(DTYPE)} layout(std430) buffer; -#define DEBUG_MODE - -#extension GL_EXT_debug_printf : enable - #include "common.glslh" ${layout_declare_tensor(B, "w", "t_cache", DTYPE, OUTPUT_STORAGE, is_scalar_array=False)} diff --git a/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_buffer.glsl b/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_buffer.glsl index c1d90fadf7e..e2d198b129f 100644 --- a/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_buffer.glsl +++ b/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_buffer.glsl @@ -12,8 +12,6 @@ ${define_active_storage_type("texture3d")} -#extension GL_EXT_debug_printf : enable - layout(std430) buffer; ${layout_declare_tensor(B, "w", "t_out", "int", "texture3d")} @@ -33,12 +31,6 @@ void main() { // Pack four 8-bit values equal to 1 into a single uint int packed = (1 << 0) | (1 << 8) | (1 << 16) | (1 << 24); - debugPrintfEXT( - "t_out[%i, %i] = %i\\n", - lpos.x, lpos.y, - packed); - - // Placeholder: just copy input to output ivec4 in_texel = ivec4(packed); imageStore(t_out, lpos, in_texel); diff --git a/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_texture3d.glsl b/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_texture3d.glsl index be6717efdaa..80e6fc27909 100644 --- a/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_texture3d.glsl +++ b/backends/vulkan/test/custom_ops/glsl/packed_int32_canvas_texture3d.glsl @@ -12,8 +12,6 @@ ${define_active_storage_type("texture2d")} -#extension GL_EXT_debug_printf : enable - layout(std430) buffer; ${layout_declare_tensor(B, "w", "t_out", "int", "texture3d")} @@ -33,12 +31,6 @@ void main() { // Pack four 8-bit values equal to 1 into a single uint int packed = (1 << 0) | (1 << 8) | (1 << 16) | (1 << 24); - debugPrintfEXT( - "t_out[%i, %i] = %i\\n", - lpos.x, lpos.y, - packed); - - // Placeholder: just copy input to output ivec4 in_texel = ivec4(packed); imageStore(t_out, lpos, in_texel);