echuraev opened a new pull request #7714:
URL: https://github.com/apache/tvm/pull/7714


   1. In case when we build runtime without ARC, we can have problems with
      memory releasing. Due to some of Objective-C methods returns
      autoreleased pointers, we should specify `autoreleasepool` blocks to
      determine life cycle of these pointers.
   2. Added fix for problem with work group size.
      Sometimes auto scheduler generates parameters when work group size
      is more than possible. And in this case we got assert from Metal
      library. Added check for this situation and it helps to avoid
      assert.
   3. Fixed memory leak problem when fill tensor by random data.
      DLManagedTensor increases reference counter in NDArray but nobody
      delete this DLManagedTensor in proper way. This is why memory which
      was allocated by NDArray was never released.
   4. Removed unnecessary retains. It is not necessary to use retain in some
      places where they were used, due to we build metal runtime without
      ARC.
   
   Thanks for contributing to TVM!   Please refer to guideline 
https://tvm.apache.org/docs/contribute/ for useful information and tips. After 
the pull request is submitted, please request code reviews from 
[Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers)
 by @ them in the pull request thread.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to