DarkSharpness commented on PR #228:
URL: https://github.com/apache/tvm-ffi/pull/228#issuecomment-3552209956
rebased cc @junrushao . Yes, your understanding is right. Since STL is not
native to tvm::ffi, type conversion will happen at each Python <-> C++. I'm not
sure where to document this. I have added a warning here in `stl.h`:
```cpp
/*!
* \file tvm/ffi/extra/stl.h
* \brief STL container support.
* \note This file is an extra extension of TVM FFI,
* which provides support for STL containers in C++ exported functions.
*
* Whenever possible, prefer using tvm/ffi/container/ implementations,
* such as `tvm::ffi::Array` and `tvm::ffi::Tuple`, over STL containers
* in exported functions for better performance and compatibility.
*/
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
