Skip to content

[build] Fix NCCL NVCC_GENCODE w/ multiple archs#8834

Merged
orionr merged 1 commit intopytorch:masterfrom
ssnl:nccl_gencode_mularch
Jun 25, 2018
Merged

[build] Fix NCCL NVCC_GENCODE w/ multiple archs#8834
orionr merged 1 commit intopytorch:masterfrom
ssnl:nccl_gencode_mularch

Conversation

@ssnl
Copy link
Collaborator

@ssnl ssnl commented Jun 24, 2018

Fixes #8729
and probably #8831

CMake automatically escapes spaces in string into \[space] for some reason. So just passing a list will work.

@ssnl ssnl requested a review from orionr as a code owner June 24, 2018 19:40
@ssnl ssnl requested review from ezyang and soumith June 24, 2018 19:41
@ssnl ssnl changed the title fix NCCL NVCC_GENCODE w/ multiple archs [build] Fix NCCL NVCC_GENCODE w/ multiple archs Jun 24, 2018
@ssnl ssnl force-pushed the nccl_gencode_mularch branch from acc37a5 to 28e063e Compare June 24, 2018 20:13
@orionr orionr merged commit 1e7fcb5 into pytorch:master Jun 25, 2018
@ssnl ssnl deleted the nccl_gencode_mularch branch June 25, 2018 15:08
@ssnl ssnl mentioned this pull request Jun 27, 2018
facebook-github-bot pushed a commit that referenced this pull request Jun 27, 2018
Summary:
Changes (were merged) in #8834 and #8829 (cc yf225 ) were lost in 9ec0a2a#diff-6997846ce6daf0c271e2db9ef0508551. This PR resubmits them.
Closes #8948

Differential Revision: D8665760

Pulled By: SsnL

fbshipit-source-id: 15514021fa79e6b908ea665dd6cb464b3ea00ab0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nccl build: nvcc fatal : redefinition of keyword 'code'

4 participants