onnx-mlir icon indicating copy to clipboard operation
onnx-mlir copied to clipboard

Refine memory management for SeqType

Open chentong319 opened this issue 2 years ago • 1 comments

KrnlSeqAlloc and KrnlSeqDealloc are introduced to support deep deallocation for the sequence type. The lowering of onnx sequence ops are changed according to the new design, which is described in docs/SequenceType.md. The new design guarantees there is no memory leak or dangling pointer for operations on onnx sequence type.

chentong319 avatar Sep 20 '22 19:09 chentong319

@AlexandreEichenberger I made changes according to your comments.

chentong319 avatar Oct 07 '22 13:10 chentong319

Jenkins Linux ppc64le Build #7294 [push] Refine memory management... started at 16:05

jenkins-droid avatar Oct 18 '22 20:10 jenkins-droid

Jenkins Linux amd64 Build #8228 [push] Refine memory management... started at 15:05

jenkins-droid avatar Oct 18 '22 20:10 jenkins-droid

Jenkins Linux s390x Build #8244 [push] Refine memory management... started at 16:05

jenkins-droid avatar Oct 18 '22 20:10 jenkins-droid

Jenkins Linux amd64 Build #8228 [push] Refine memory management... passed after 1 hr 4 min

jenkins-droid avatar Oct 18 '22 21:10 jenkins-droid

Jenkins Linux s390x Build #8244 [push] Refine memory management... passed after 1 hr 26 min

jenkins-droid avatar Oct 18 '22 21:10 jenkins-droid

Jenkins Linux ppc64le Build #7294 [push] Refine memory management... passed after 1 hr 37 min

jenkins-droid avatar Oct 18 '22 21:10 jenkins-droid