chunit-quic commented on a change in pull request #9723: URL: https://github.com/apache/tvm/pull/9723#discussion_r769717976
########## File path: src/printer/relay_text_printer.cc ########## @@ -389,12 +389,21 @@ Doc RelayTextPrinter::VisitExpr_(const TupleNode* op) { if (op->fields.size() == 1) { doc << ","; } - return doc << ")"; + doc << ")"; + if (op->span.defined()) { Review comment: Hi @jroesch It seems that there is a more suitable printer for this part. Would you mind to share that feature with me? Sorry that I just follow the existing format without checking it more carefully. Once we have conclusion about which one should be used this time, I could try to modify my current code. :D -- 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-unsubscr...@tvm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org