Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CUDA 9.0 deprecated non-sync shuffle instructions #136

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ProfFan
Copy link

@ProfFan ProfFan commented Oct 17, 2018

This PR fixes the following Warning:

ptxas /tmp/tmpxft_000010c5_00000000-5_WarpingSolver.ptx, line 677;
warning : Instruction 'shfl' without '.sync' is deprecated since PTX ISA version 6.0 and
will be discontinued in a future PTX ISA version

Tested on my Mac with LLVM 6.0 and latest Opt/Terra

I did not see performance regressions with the demo example applications.

Best,
Fan

@Mx7f
Copy link
Collaborator

Mx7f commented Oct 17, 2018

Thanks! Unfortunately the .sync. operators weren't introduced until after CUDA 7.5 (Opt's currently recommended version). So either we need to detect the CUDA version and switch what code we use, or just wait to merge this until we get pre-built terra binaries compatible with CUDA 9 (terralang/terra#299 (comment)).

@ProfFan
Copy link
Author

ProfFan commented Jun 4, 2019

Bump @Mx7f :)

Sign up for free to 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.

None yet

2 participants