wweic commented on a change in pull request #4643: [REFACTOR] Replace TensorObj
and TensorValue with NDArray
URL: https://github.com/apache/incubator-tvm/pull/4643#discussion_r364077225
##########
File path: include/tvm/relay/interpreter.h
##########
@@ -65,39 +61,21 @@ class Value;
* \param target Compiler target flag to compile the functions on the context.
* \return A function that takes in an expression and returns a value.
*/
-runtime::TypedPackedFunc<Value(Expr)>
+runtime::TypedPackedFunc<ObjectRef(Expr)>
CreateInterpreter(Module mod, DLContext context, Target target);
-/*! \brief The base container type of Relay values. */
-class ValueNode : public RelayNode {
- public:
- static constexpr const char* _type_key = "relay.Value";
- TVM_DECLARE_BASE_OBJECT_INFO(ValueNode, RelayNode);
-};
-
-class Value : public ObjectRef {
- public:
- Value() {}
- explicit Value(ObjectPtr<Object> n) : ObjectRef(n) {}
- const ValueNode* operator->() const {
- return static_cast<const ValueNode*>(get());
- }
-
- using ContainerType = ValueNode;
-};
-
/*! \brief A Relay closure, i.e a scope and a function. */
class Closure;
/*! \brief The container type of Closures. */
-class ClosureNode : public ValueNode {
+class ClosureNode : public Object {
Review comment:
```suggestion
class ClosureObj : public Object {
```
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
