tvm-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <>
Subject [GitHub] [incubator-tvm] zhiics commented on issue #5276: [REFACTOR][IR] Move to runtime::String
Date Wed, 08 Apr 2020 06:15:00 GMT
zhiics commented on issue #5276: [REFACTOR][IR] Move to runtime::String
   @tqchen One problem of replacing the default container StringImm with runtime String is
that some of the IR nodes are using PrimExpr which could be both StringImm or IntImm (they
are implicitly converted from std::string, integers, or floats). For example,
   This would fail if we pass string and default it to runtime String, i.e.
   Should we explicitly convert it from runtime String to StringImm in this case either before
the constructor in Python or in the PackedFunc in the C++? This looks pretty ugly. Could you
suggest some better solutions?

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:

With regards,
Apache Git Services

View raw message