Skip to content

Commit f7e7064

Browse files
authored
[mlir][SVE] Fix memory leaks in integration tests (#86488)
Buffers are no longer deallocated by One-Shot Bufferize - this is now done by a separate buffer deallocation pass. In order to see the leaks in SVE integration tests, use the following CMake flags (enables the address sanitizer and SVE integration tests): -DLLVM_USE_SANITIZER="Address" -DMLIR_INCLUDE_INTEGRATION_TESTS=On -DMLIR_RUN_ARM_SVE_TESTS=On Follow-up for #85366
1 parent b331664 commit f7e7064

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

mlir/test/Integration/Dialect/Linalg/CPU/ArmSVE/fill-1d.mlir

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// RUN: mlir-opt %s -transform-interpreter -test-transform-dialect-erase-schedule -lower-vector-mask -one-shot-bufferize -test-lower-to-llvm | \
1+
// RUN: mlir-opt %s -transform-interpreter -test-transform-dialect-erase-schedule -lower-vector-mask -one-shot-bufferize -buffer-deallocation-pipeline -test-lower-to-llvm | \
22
// RUN: %mcr_aarch64_cmd -e=entry -entry-point-result=void --march=aarch64 --mattr="+sve" -shared-libs=%mlir_runner_utils,%mlir_c_runner_utils | \
33
// RUN: FileCheck %s
44

mlir/test/Integration/Dialect/Linalg/CPU/ArmSVE/matmul.mlir

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// DEFINE: %{compile} = mlir-opt %s \
22
// DEFINE: -transform-interpreter -test-transform-dialect-erase-schedule \
3-
// DEFINE: -one-shot-bufferize -func-bufferize -cse -canonicalize -convert-vector-to-scf -arm-sve-legalize-vector-storage \
3+
// DEFINE: -one-shot-bufferize="bufferize-function-boundaries" -buffer-deallocation-pipeline -cse -canonicalize -convert-vector-to-scf -arm-sve-legalize-vector-storage \
44
// DEFINE: -convert-vector-to-llvm="enable-arm-sve" -test-lower-to-llvm -o %t
55
// DEFINE: %{entry_point} = matmul_f32
66
// DEFINE: %{run} = %mcr_aarch64_cmd %t -e %{entry_point} -entry-point-result=void --march=aarch64 --mattr="+sve"\

0 commit comments

Comments
 (0)