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 2020/11/17 23:10:44 UTC

[GitHub] [incubator-tvm] jroesch commented on a change in pull request #6874: [Diagnostics][WIP] Enable AnnotateSpans by default, add environment variable for controlling top-level.

jroesch commented on a change in pull request #6874:
URL: https://github.com/apache/incubator-tvm/pull/6874#discussion_r525586496



##########
File path: src/parser/parser.cc
##########
@@ -605,30 +605,42 @@ class Parser {
     return ast;
   }
 
+  struct MetaRef {

Review comment:
       Yes they fix a bug in the parsing which prevents this from working on all source files. I was going to turn on annotate spans for all models. 




----------------------------------------------------------------
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:
users@infra.apache.org