Skip to content

[Build] Explicitly link to cublasLt if it exists - #4776

Merged
masahi merged 2 commits into
apache:masterfrom
soiferj:cublaslt
Jan 27, 2020
Merged

[Build] Explicitly link to cublasLt if it exists#4776
masahi merged 2 commits into
apache:masterfrom
soiferj:cublaslt

Conversation

@soiferj

Copy link
Copy Markdown
Contributor

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?

@masahi

masahi commented Jan 25, 2020

Copy link
Copy Markdown
Member

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

@soiferj

soiferj commented Jan 25, 2020 via email

Copy link
Copy Markdown
ContributorAuthor

@soiferjsoiferj changed the title [Build] Explicitly link to cublasLt[Build] Explicitly link to cublasLt if it existsJan 27, 2020
@soiferj

Copy link
Copy Markdown
ContributorAuthor

@masahi fixed. Would you mind taking another look?

@masahi
masahi merged commit 00ec7f9 into apache:masterJan 27, 2020
@masahi

Copy link
Copy Markdown
Member

thanks @soiferj

@soiferj
soiferj deleted the cublaslt branch January 27, 2020 22:38
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 26, 2020
* Explicitly link to cublasLt
* Only link cublasLt if it's found
Co-authored-by: Jon Soifer <jonso@microsoft.com>
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 28, 2020
* Explicitly link to cublasLt
* Only link cublasLt if it's found
Co-authored-by: Jon Soifer <jonso@microsoft.com>
zhiics pushed a commit to neo-ai/tvm that referenced this pull request Mar 2, 2020
* Explicitly link to cublasLt
* Only link cublasLt if it's found
Co-authored-by: Jon Soifer <jonso@microsoft.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@soiferj@masahi@jonso4