Lunderberg commented on pull request #10272:
URL: https://github.com/apache/tvm/pull/10272#issuecomment-1042122406


   No strong preferences from me either way.  #10262 is consistent with how 
string constants are defined elsewhere in TVM, but my preference is to keep 
them close to the context where they're used.
   
   Since this one is still waiting on CI, I'd go with #10262 and closing #10272.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to