Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Merge NVC++ CMake changes into main #244

Merged
merged 16 commits into from
Feb 12, 2022
Merged

Merge NVC++ CMake changes into main #244

merged 16 commits into from
Feb 12, 2022

Conversation

wmaxey
Copy link
Member

@wmaxey wmaxey commented Feb 8, 2022

These changes remove the need to use the upstream LLVM project on Windows for lit configuration.

@wmaxey wmaxey self-assigned this Feb 8, 2022
@wmaxey wmaxey requested review from griwes and brycelelbach February 8, 2022 02:30
@wmaxey
Copy link
Member Author

wmaxey commented Feb 10, 2022

This changes quite a bit of docker stuff, one thing I couldn't quite figure out was the default NVCC build where no architecture was specified. Prior to these changes it wasn't throwing up on tests that should have been unsupported.

To fix that behavior I changed config.py to launch with NVCC defaults and all newer SMs disabled.

3e28816

@wmaxey wmaxey force-pushed the feature/nvcxx-cmake-rebase branch 2 times, most recently from 8406bd1 to 8e03396 Compare February 10, 2022 19:12
@wmaxey wmaxey added the testing: internal ci passed Passed internal NVIDIA CI (DVS). label Feb 11, 2022
@wmaxey wmaxey force-pushed the feature/nvcxx-cmake-rebase branch from 8e03396 to cf54cb7 Compare February 11, 2022 23:16
griwes and others added 14 commits February 11, 2022 17:35
…e configuration.

Right now it all seems to work, just need adjustment in
perform_tests.bash and to eventually fix all the issues with running
some of the test suites in some of the configurations.
@wmaxey wmaxey force-pushed the feature/nvcxx-cmake-rebase branch from b264189 to 17a3055 Compare February 12, 2022 01:38
@wmaxey wmaxey merged commit 60a1fd1 into main Feb 12, 2022
@wmaxey wmaxey deleted the feature/nvcxx-cmake-rebase branch February 12, 2022 05:07
@wmaxey wmaxey added this to the 1.8.0 milestone Feb 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing: internal ci passed Passed internal NVIDIA CI (DVS).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants