csullivan commented on code in PR #10910:
URL: https://github.com/apache/tvm/pull/10910#discussion_r844106727
##########
src/runtime/hexagon/hexagon/hexagon_device_api_v2.h:
##########
@@ -125,8 +126,23 @@ class HexagonDeviceAPIv2 final : public DeviceAPI {
TVMStreamHandle stream) final;
private:
- //! Lookup table for the HexagonBuffer managing a workspace allocation.
- std::unordered_map<void*, HexagonBuffer*> workspace_allocations_;
+ /*! \brief Helper to allocate a HexagonBuffer and register the result
+ * in the owned buffer map.
+ * \return Raw data storage managed by the hexagon buffer
+ */
+ template <typename... Args>
+ void* AllocateHexagonBuffer(Args&&... args) {
+ auto buf = std::unique_ptr<HexagonBuffer>(new
HexagonBuffer(std::forward<Args>(args)...));
Review Comment:
Alas my configuration for hexagon was giving false negatives on finding
std::make_unique in the std. I had intended to circle back on this one before
pushing but forgot, so thank you for the reminder!
--
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]
