Skip to content

Commit

Permalink
Add support for Volta GPUs (sm_70)
Browse files Browse the repository at this point in the history
  • Loading branch information
@fwyzard
fwyzard committed Sep 28, 2018
1 parent 0ac9f37 commit 89c8f32
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions cuda-toolfile.spec
View file
Expand Up @@ -37,6 +37,7 @@ cat << \EOF_TOOLFILE >%{i}/etc/scram.d/cuda.xml
<flags CUDA_FLAGS="-gencode arch=compute_35,code=sm_35"/>
<flags CUDA_FLAGS="-gencode arch=compute_50,code=sm_50"/>
<flags CUDA_FLAGS="-gencode arch=compute_61,code=sm_61"/>

This comment has been minimized.

Sign in to view

Copy link
@smuzaffar

smuzaffar Dec 12, 2018

Contributor

@fwyzard , should we replace sm_61 with sm_60 to get it working for P100?
https://github.com/tpruvot/ccminer/wiki/Compatibility

This comment has been minimized.

Sign in to view

Copy link
@fwyzard

fwyzard Dec 12, 2018

Author Contributor

That would break it for the gtx1080 cards.
To support both, we need to a new line for sm60.

This comment has been minimized.

Sign in to view

Copy link
@smuzaffar

smuzaffar Dec 13, 2018

Contributor

@fwyzard , ok after adding sm_60 I can run test_detid unittest on P100 techlab machine (both inside singularity and outside).
<flags CUDA_FLAGS="-gencode arch=compute_70,code=sm_70"/>
<flags CUDA_FLAGS="-O3 -std=c++14 --expt-relaxed-constexpr --expt-extended-lambda"/>
<flags CUDA_HOST_REM_CXXFLAGS="-std=%"/>
<flags CUDA_HOST_REM_CXXFLAGS="%potentially-evaluated-expression"/>
Expand Down

0 comments on commit 89c8f32

Please sign in to comment.