u99127 commented on pull request #30:
URL: https://github.com/apache/tvm-rfcs/pull/30#issuecomment-920701756


   > Can you address the common usage of environment variables for 
configuration, why that wasn't considered, and the potential for future work?
   
   I think the usage of environment variables for configuration of this kind in 
an AoT compilation flow with tvmc is quite debatable with the kinds of things 
it takes. .
   
   Personally I think its cleaner to consider this directly in configuration 
files of this kind. 
   
   I have had a read through and this looks pretty good LGTM
   
   @leandron  could you please take a look ? 
   


-- 
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


Reply via email to