[Mlir-commits] [mlir] [mlir][linalg, tosa] Fix memory leaks in integration tests (PR #85366)

Matthias Springer llvmlistbot at llvm.org
Fri Mar 15 20:22:41 PDT 2024


matthias-springer wrote:

There are a few tests in `test/Integration/Dialect/Linalg` that have `-buffer-deallocation-pipeline`.

> Is it worth updating existing tests to always use this to reduce the likelihood of leaks happening in the future?

Yes, I'm in the process of fixing all memory leaks at the moment and this PR will bring down the number of leaking tests to 5 or so. The remaining ones are sparse compiler tests and they need a different fix (but they will eventually also use the buffer deallocation pipeline). To check for leaks, I built MLIR with `-DLLVM_USE_SANITIZER="Address"` and ran `check-mlir`. Ideally, we'd have a build bot that runs with ASAN.


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


More information about the Mlir-commits mailing list