You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/08/18 12:18:58 UTC

[GitHub] [tvm] masahi commented on a diff in pull request #12489: [TIR] Support AllocateConst nodes in TensorIR scheduling flow

masahi commented on code in PR #12489:
URL: https://github.com/apache/tvm/pull/12489#discussion_r949057460


##########
src/printer/tvmscript_printer.cc:
##########
@@ -416,26 +416,6 @@ class TVMScriptPrinter : public StmtFunctor<Doc(const Stmt&)>,
   }
 };
 
-/*!
- * \brief special method to print NDArray in TIR
- * \param arr the NDArray to be printed
- * \param os the output stream where the NDArray will be printed to
- */
-template <typename T>
-void NDArrayToTIR(::tvm::runtime::NDArray arr, std::ostream& os) {

Review Comment:
   I removed this because otherwise it tries to write a huge constant array to a string. I don't think it is useful to print out the numerical value of constants.



-- 
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: commits-unsubscribe@tvm.apache.org

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