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

[llvmlistbot at llvm.org]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Davide Grohmann (davidegrohmann)

<details>
<summary>Changes</summary>

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

---
Full diff: https://github.com/llvm/llvm-project/pull/200137.diff


1 Files Affected:

- (modified) mlir/lib/Target/SPIRV/Deserialization/DeserializeOps.cpp (-3) 


``````````diff
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,

``````````

</details>


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