[Mlir-commits] [mlir] [mlir][spirv] Remove unnecessary assertion (PR #200137)

Davide Grohmann llvmlistbot at llvm.org
Thu May 28 02:06:45 PDT 2026


https://github.com/davidegrohmann created https://github.com/llvm/llvm-project/pull/200137

The use of the variable in the assertion was causing a build failure when compiling with assertion off and hence the variable becomes unused.

>From 80fecebabfe2d6d60a5c0e974041e7c01bdccba9 Mon Sep 17 00:00:00 2001
From: Davide Grohmann <davide.grohmann at arm.com>
Date: Thu, 28 May 2026 11:01:02 +0200
Subject: [PATCH] [mlir][spirv] Remove unnecessary assertion

The use of the variable in the assertion was causing a build failure
when compiling with assertion off and hence the variable becomes
unused.

Signed-off-by: Davide Grohmann <davide.grohmann at arm.com>
Change-Id: I39737ea8066b7b0349ce613d40bb4b3809663ea6
---
 mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp b/mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp
index e3fa2d48793d7..0e6d199eeabf0 100644
--- a/mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp
+++ b/mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp
@@ -417,9 +417,6 @@ spirv::Deserializer::processDebugInfoExtInst(ArrayRef<uint32_t> operands,
                      "<id>, result <id>, set <id> and instruction opcode");
   }
 
-  StringRef &extensionSetName = extendedInstSets[operands[2]];
-  assert(extensionSetName == extDebugInfo);
-
   Type resultType = getType(operands[0]);
   if (!resultType || !isVoidType(resultType))
     return emitError(unknownLoc,



More information about the Mlir-commits mailing list