[Mlir-commits] [mlir] [mlir] [bufferize] fix bufferize deallocation error in nest symbol table (PR #98476)

Matthias Springer llvmlistbot at llvm.org
Sun Jul 14 11:56:08 PDT 2024


================
@@ -132,27 +132,30 @@ struct BufferizationToMemRefPass
       return;
     }
 
-    func::FuncOp helperFuncOp;
+    llvm::DenseMap<Operation *, func::FuncOp> deallocHelperFuncMap;
----------------
matthias-springer wrote:

nit: Can you add a type alias for this? (With a comment that the `Operation*` is a symbol table.) I think it would improve the readability a bit.

```c++
using DeallocHelperMap = llvm::DenseMap<Operation *, func::FuncOp>;
```


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


More information about the Mlir-commits mailing list