[Mlir-commits] [mlir] [mlir][SPIR-V] Clear value and block ID maps after each function (PR #237171)

[llvmlistbot at llvm.org]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir-spirv

Author: Arseniy Obolenskiy (aobolensk)

<details>
<summary>Changes</summary>

SSA values and blocks are function local, so old entries only grow the maps

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


1 Files Affected:

- (modified) mlir/lib/Target/SPIRV/Serialization/SerializeOps.cpp (+2) 


``````````diff
diff --git a/mlir/lib/Target/SPIRV/Serialization/SerializeOps.cpp b/mlir/lib/Target/SPIRV/Serialization/SerializeOps.cpp
index 254470eb5934f3..a665e326156af7 100644
--- a/mlir/lib/Target/SPIRV/Serialization/SerializeOps.cpp
+++ b/mlir/lib/Target/SPIRV/Serialization/SerializeOps.cpp
@@ -432,6 +432,8 @@ LogicalResult Serializer::processFuncOp(spirv::FuncOp op) {
   functions.append(functionBody.begin(), functionBody.end());
   functionHeader.clear();
   functionBody.clear();
+  valueIDMap.clear();
+  blockIDMap.clear();
 
   return success();
 }

``````````

</details>


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