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

Fix config when only non-CDPv1 arches are enabled. #1109

Merged

Conversation

alliepiper
Copy link
Collaborator

Allows things like ci/build_cub.sh -arch 90 to work as-expected.

Fixes #219,

Description

Adds logic to skip adding RDC targets when the RDC arch list is empty.

@alliepiper alliepiper requested review from a team as code owners November 15, 2023 18:08
@alliepiper alliepiper requested review from wmaxey and elstehle and removed request for a team November 15, 2023 18:08
@alliepiper alliepiper force-pushed the check_rdc_arch_list_for_cdp_targets branch from c91fa4f to 18c6031 Compare November 15, 2023 18:30
Allows things like `ci/build_cub.sh -arch 90` to work as-expected.

Fixes NVIDIA#219.
@alliepiper alliepiper force-pushed the check_rdc_arch_list_for_cdp_targets branch from 18c6031 to 4a1e64f Compare November 15, 2023 18:30
@alliepiper alliepiper merged commit 931dc67 into NVIDIA:main Nov 15, 2023
518 checks passed
@alliepiper alliepiper deleted the check_rdc_arch_list_for_cdp_targets branch November 15, 2023 19:41
@wmaxey
Copy link
Member

wmaxey commented Nov 15, 2023

Can this be backported to branch/2.3.x?

@wmaxey wmaxey added the backport branch/2.3.x For backporting to the 2.3.x release branch label Nov 15, 2023
@wmaxey
Copy link
Member

wmaxey commented Nov 15, 2023

/backport

Copy link
Contributor

Successfully created backport PR for branch/2.3.x:

@miscco miscco added backport done This PR has been backported to the relevant branch and removed backport branch/2.3.x For backporting to the 2.3.x release branch labels Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport done This PR has been backported to the relevant branch
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]: Thrust/CUB should check that their CUDA_ARCHITECTURES_RDC list is non-empty before adding RDC targets.
5 participants