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/05/22 16:39:15 UTC

[GitHub] [incubator-tvm] tqchen opened a new pull request #5652: [TIR][BUILD] Remove buffer params from pass config.

tqchen opened a new pull request #5652:
URL: https://github.com/apache/incubator-tvm/pull/5652


   Buffer configurations can be passed during construction
   and does not need to be part of the build config.
   
   This is a refactor step to simplify the buildconfig for the PassContext migration.
   
   


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



[GitHub] [incubator-tvm] roastduck commented on pull request #5652: [TIR][BUILD] Remove buffer params from pass config.

Posted by GitBox <gi...@apache.org>.
roastduck commented on pull request #5652:
URL: https://github.com/apache/incubator-tvm/pull/5652#issuecomment-633442386


   I noticed that some buffers lost their name in the IR dump or CUDA dump after merging this PR. For example, `x_1` in `softmax` for CUDA becomes `buffer_2`. Is is change in purpose?


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



[GitHub] [incubator-tvm] tqchen commented on pull request #5652: [TIR][BUILD] Remove buffer params from pass config.

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #5652:
URL: https://github.com/apache/incubator-tvm/pull/5652#issuecomment-632791420


   cc @ZihengJiang @zhiics 


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



[GitHub] [incubator-tvm] tqchen merged pull request #5652: [TIR][BUILD] Remove buffer params from pass config.

Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #5652:
URL: https://github.com/apache/incubator-tvm/pull/5652


   


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