This is an archive of the discontinued LLVM Phabricator instance.

Differential D130914

[mlir][sparse] fix asan leak of two sparse vectors
ClosedPublic

Authored by aartbik on Aug 1 2022, 9:23 AM.

Details

Reviewers
bixia
Peiming
Commits
rG3b9bee1651af: [mlir][sparse] fix asan leak of two sparse vectors

Diff Detail

Event Timeline

aartbik created this revision.Aug 1 2022, 9:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 9:23 AM
Herald added subscribers: anlunx, bzcheeseman, sdasgup3 and 19 others. · View Herald Transcript
aartbik requested review of this revision.Aug 1 2022, 9:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 9:23 AM
Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald Transcript
aartbik added reviewers: bixia, Peiming.Aug 1 2022, 9:24 AM
Harbormaster completed remote builds in B178590: Diff 449042.Aug 1 2022, 10:48 AM
Peiming accepted this revision.Aug 1 2022, 10:54 AM

LGTM

This revision is now accepted and ready to land.Aug 1 2022, 10:54 AM
Closed by commit rG3b9bee1651af: [mlir][sparse] fix asan leak of two sparse vectors (authored by aartbik). · Explain WhyAug 1 2022, 10:55 AM
This revision was automatically updated to reflect the committed changes.
aartbik added a commit: rG3b9bee1651af: [mlir][sparse] fix asan leak of two sparse vectors.

Revision Contents

PathSize
mlir/
test/
Integration/
Dialect/
SparseTensor/
CPU/
2 lines

Diff 449073

mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_binary.mlir

Show First 20 Lines • Show All 531 Lines • ▼ Show 20 Lines func.func @entry() {
call @dump_mat_4x4(%8) : (tensor<4x4xf64, #DCSR>) -> () call @dump_mat_4x4(%8) : (tensor<4x4xf64, #DCSR>) -> ()
call @dump_mat_4x4(%9) : (tensor<4x4xf64, #DCSR>) -> () call @dump_mat_4x4(%9) : (tensor<4x4xf64, #DCSR>) -> ()
call @dump_mat_4x4_i8(%10) : (tensor<4x4xi8, #DCSR>) -> () call @dump_mat_4x4_i8(%10) : (tensor<4x4xi8, #DCSR>) -> ()
call @dump_mat_4x4(%11) : (tensor<4x4xf64, #DCSR>) -> () call @dump_mat_4x4(%11) : (tensor<4x4xf64, #DCSR>) -> ()
// Release the resources. // Release the resources.
bufferization.dealloc_tensor %sv1 : tensor<?xf64, #SparseVector> bufferization.dealloc_tensor %sv1 : tensor<?xf64, #SparseVector>
bufferization.dealloc_tensor %sv2 : tensor<?xf64, #SparseVector> bufferization.dealloc_tensor %sv2 : tensor<?xf64, #SparseVector>
bufferization.dealloc_tensor %sv1_si : tensor<?xi32, #SparseVector>
bufferization.dealloc_tensor %sv2_si : tensor<?xi32, #SparseVector>
bufferization.dealloc_tensor %sm1 : tensor<?x?xf64, #DCSR> bufferization.dealloc_tensor %sm1 : tensor<?x?xf64, #DCSR>
bufferization.dealloc_tensor %sm2 : tensor<?x?xf64, #DCSR> bufferization.dealloc_tensor %sm2 : tensor<?x?xf64, #DCSR>
bufferization.dealloc_tensor %sm3 : tensor<4x4xf64, #DCSR> bufferization.dealloc_tensor %sm3 : tensor<4x4xf64, #DCSR>
bufferization.dealloc_tensor %sm4 : tensor<4x4xf64, #DCSR> bufferization.dealloc_tensor %sm4 : tensor<4x4xf64, #DCSR>
bufferization.dealloc_tensor %0 : tensor<?xi32, #SparseVector> bufferization.dealloc_tensor %0 : tensor<?xi32, #SparseVector>
bufferization.dealloc_tensor %1 : tensor<?xf64, #SparseVector> bufferization.dealloc_tensor %1 : tensor<?xf64, #SparseVector>
bufferization.dealloc_tensor %2 : tensor<?xf64, #SparseVector> bufferization.dealloc_tensor %2 : tensor<?xf64, #SparseVector>
bufferization.dealloc_tensor %3 : tensor<?xi32, #SparseVector> bufferization.dealloc_tensor %3 : tensor<?xi32, #SparseVector>
Show All 10 Lines