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

Cullen Rhodes llvmlistbot at llvm.org
Fri Mar 15 02:50:13 PDT 2024


c-rhodes wrote:

> This changed around September last year. We posted a [PSA](https://discourse.llvm.org/t/psa-bufferization-new-buffer-deallocation-pipeline/73375) and migration guide but you may have missed it.
> 

Thanks for link.

> Btw, I ran `check-mlir` with the address sanitizer today and I did not see any failing/leaking ArmSME tests.

Great thanks. Grepping on the existing tests I see there's only a single one using `-buffer-deallocation-pipeline`. With so few tests using it I can see how it could be missed this is required, my approach at least is usually to look at what other tests are doing. Is it worth updating existing tests to always use this to reduce the likelihood of leaks happening in the future?

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


More information about the Mlir-commits mailing list