Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Add magma build for CUDA 11#461

Merged
seemethere merged 2 commits intopytorch:masterfrom
zasdfgbnm:cuda11-magma
Aug 3, 2020
Merged

Add magma build for CUDA 11#461
seemethere merged 2 commits intopytorch:masterfrom
zasdfgbnm:cuda11-magma

Conversation

@zasdfgbnm
Copy link
Copy Markdown
Contributor

Copied from 10.2 with additional changes.

  • cmakelists.patch:
    • add sm80 and compute80
    • disable magma sparse
  • magma/magma-cuda110/cudaPointerAttributes.patch


+ if ( ${GPU_TARGET} MATCHES "All")
+ set( MIN_ARCH 370)
+ SET( NV_SM -gencode arch=compute_37,code=sm_37 -gencode arch=compute_50,code=sm_50 -gencode arch=compute_60,code=sm_60 -gencode arch=compute_70,code=sm_70 )
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, don't you want to add compute_80 here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I remembered I added it here but looks like I was wrong. Fixed now.

Comment on lines +7 to +8
-option( USE_FORTRAN "Fortran is required for some tester checks, but can be disabled with reduced functionality" ON )
+option( USE_FORTRAN "Fortran is required for some tester checks, but can be disabled with reduced functionality" OFF )
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this change is needed, if cmake is called with -DUSE_FORTRAN=OFF?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know, this part is copied from CUDA 10.2 patch.

@zasdfgbnm zasdfgbnm changed the title [WIP] Add magma build for CUDA 11 Add magma build for CUDA 11 Jun 25, 2020
@dawonn-haval
Copy link
Copy Markdown

Any guess when this might be merged?

@zasdfgbnm
Copy link
Copy Markdown
Contributor Author

@dawonn-haval I believe it will be merged soon after 1.6 release of pytorch

@seemethere seemethere merged commit 859513c into pytorch:master Aug 3, 2020
@zasdfgbnm zasdfgbnm deleted the cuda11-magma branch August 3, 2020 18:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants