From ae1ee505a4eb4d798e5ea9a16dbc402596b82907 Mon Sep 17 00:00:00 2001 From: Stephen Jia Date: Mon, 13 Oct 2025 14:29:02 -0400 Subject: [PATCH] [ET-VK][ez] Fix unused variable Summary: Title says it all. Also add `-Werror=unused-variable` to compile flags when building Vulkan to guard against this in the future. --- backends/vulkan/CMakeLists.txt | 1 + backends/vulkan/runtime/graph/ops/impl/BinaryOp.cpp | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/backends/vulkan/CMakeLists.txt b/backends/vulkan/CMakeLists.txt index d9acde79ecf..fe25d62d63a 100644 --- a/backends/vulkan/CMakeLists.txt +++ b/backends/vulkan/CMakeLists.txt @@ -52,6 +52,7 @@ set(COMMON_INCLUDES set(VULKAN_CXX_FLAGS "-fexceptions") list(APPEND VULKAN_CXX_FLAGS "-DUSE_VULKAN_WRAPPER") list(APPEND VULKAN_CXX_FLAGS "-DUSE_VULKAN_VOLK") +list(APPEND VULKAN_CXX_FLAGS "-Werror=unused-variable") # vulkan API files diff --git a/backends/vulkan/runtime/graph/ops/impl/BinaryOp.cpp b/backends/vulkan/runtime/graph/ops/impl/BinaryOp.cpp index 9575ca0dcdd..eaf05ff14bc 100644 --- a/backends/vulkan/runtime/graph/ops/impl/BinaryOp.cpp +++ b/backends/vulkan/runtime/graph/ops/impl/BinaryOp.cpp @@ -161,7 +161,6 @@ void add_binary_op_buffer_node( std::string kernel_name("binary_"); kernel_name.reserve(kShaderNameReserve); std::string op = op_name; - int clamp_type = remove_clamp_from_name(op); kernel_name += op; add_storage_type_suffix(kernel_name, graph.storage_type_of(out));