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/01/24 22:45:28 UTC

[GitHub] [incubator-tvm] soiferj opened a new pull request #4776: [Build] Explicitly link to cublasLt

soiferj opened a new pull request #4776: [Build] Explicitly link to cublasLt
URL: https://github.com/apache/incubator-tvm/pull/4776
 
 
   Currently, the Windows build fails due to the linker not being able to find cublasLt functions. This change explicitly links to cublasLt.
   
   @Laurawly @masahi @Hzfengsy would you be able to 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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-tvm] masahi edited a comment on issue #4776: [Build] Explicitly link to cublasLt

Posted by GitBox <gi...@apache.org>.
masahi edited a comment on issue #4776: [Build] Explicitly link to cublasLt
URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578367047
 
 
   @soiferj the CI is at CUDA 10.0, so it doesn't have cublaslt lib. We will be updating the CI soon #4753 #4756, we can also discuss if we want to go CUDA 10.1 (but it may be too big a change)

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


With regards,
Apache Git Services

[GitHub] [incubator-tvm] soiferj commented on issue #4776: [Build] Explicitly link to cublasLt if it exists

Posted by GitBox <gi...@apache.org>.
soiferj commented on issue #4776: [Build] Explicitly link to cublasLt if it exists
URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578902685
 
 
   @masahi fixed. Would you mind taking another 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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-tvm] masahi commented on issue #4776: [Build] Explicitly link to cublasLt

Posted by GitBox <gi...@apache.org>.
masahi commented on issue #4776: [Build] Explicitly link to cublasLt
URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578367047
 
 
   @soiferj the CI is at CUDA 10.0, so it doesn't have cublaslt lib.

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


With regards,
Apache Git Services

[GitHub] [incubator-tvm] masahi commented on issue #4776: [Build] Explicitly link to cublasLt if it exists

Posted by GitBox <gi...@apache.org>.
masahi commented on issue #4776: [Build] Explicitly link to cublasLt if it exists
URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578985825
 
 
   thanks @soiferj 

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


With regards,
Apache Git Services

[GitHub] [incubator-tvm] masahi merged pull request #4776: [Build] Explicitly link to cublasLt if it exists

Posted by GitBox <gi...@apache.org>.
masahi merged pull request #4776: [Build] Explicitly link to cublasLt if it exists
URL: https://github.com/apache/incubator-tvm/pull/4776
 
 
   

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


With regards,
Apache Git Services

[GitHub] [incubator-tvm] soiferj commented on issue #4776: [Build] Explicitly link to cublasLt

Posted by GitBox <gi...@apache.org>.
soiferj commented on issue #4776: [Build] Explicitly link to cublasLt
URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578373537
 
 
   Got it. In that case, I’ll update the logic to only link if it found the library.
   
   
   ________________________________
   From: masahi <no...@github.com>
   Sent: Friday, January 24, 2020 8:14 PM
   To: apache/incubator-tvm
   Cc: Jon Soifer; Mention
   Subject: Re: [apache/incubator-tvm] [Build] Explicitly link to cublasLt (#4776)
   
   
   @soiferj<https://github.com/soiferj> the CI is at CUDA 10.0, so it doesn't have cublaslt lib.
   
   —
   You are receiving this because you were mentioned.
   Reply to this email directly, view it on GitHub<https://github.com/apache/incubator-tvm/pull/4776?email_source=notifications&email_token=AA26773C5W6X54VIZ3HKCELQ7OOBZA5CNFSM4KLNCYOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ4S4RY#issuecomment-578367047>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AA2677Y4MA56HLVDNJ43LCLQ7OOBZANCNFSM4KLNCYOA>.
   

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


With regards,
Apache Git Services