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 2021/11/26 08:14:59 UTC

[GitHub] [tvm] masahi commented on pull request #9593: Concatenate kernel optimization for CPU target

masahi commented on pull request #9593:
URL: https://github.com/apache/tvm/pull/9593#issuecomment-979773443


   Per offline discussion:
   
   * `concatenate_2d` should be removed (it was just a quick hack for DLRM benchmarking)
   * The change should only be applied to x86 for now. 
   * We can keep both the original and new x86 concat impl and let AutoTVM decide the better one for a given workload. The only question to address here is which one to choose by default.
   


-- 
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-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org