Skip to content

Add an nvcc test which compiles all the headers at once - #1519

Merged
Stephan T. Lavavej (StephanTLavavej) merged 18 commits into
microsoft:masterfrom
cbezault:nvcc
Dec 17, 2020
Merged

Add an nvcc test which compiles all the headers at once#1519
Stephan T. Lavavej (StephanTLavavej) merged 18 commits into
microsoft:masterfrom
cbezault:nvcc

Conversation

@cbezault

@cbezault Curtis J Bezault (cbezault) commented Dec 9, 2020

Copy link
Copy Markdown
Contributor

Fixes #639

This PR also changes the mechanism by which we detect tests. We now recognize any file ending with compile.pass.cpp or test.cpp under tr1/tests and std/tests as a test. The test.lst files are now only for internal usage.

@cbezault
Curtis J Bezault (cbezault) requested a review from a team as a code owner December 9, 2020 00:09
@cbezault Curtis J Bezault (cbezault) changed the title Add an nvcc test run to the include each header alone test Add an nvcc test which compiles all the headers at once Dec 9, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these "requested changes" may actually be "requested explanation" ;)

Comment thread tests/std/tests/GH_000639_nvcc_include_all/env.lst Outdated
Comment thread tests/std/tests/GH_000639_nvcc_include_all/env.lst
Comment thread tests/std/tests/GH_000639_nvcc_include_all/env.lst
Comment thread tests/std/lit.site.cfg.in
Comment thread tests/std/lit.site.cfg.in
Comment thread tests/tr1/lit.site.cfg.in
Comment thread tests/std/tests/GH_000639_nvcc_include_all/env.lst
Comment thread tests/std/tests/GH_000639_nvcc_include_all/env.lst Outdated
@StephanTLavavej

Copy link
Copy Markdown
Member

Everything looks great. I've pushed a merge with master, no conflicts.

@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for figuring out the necessary magic to improve our CUDA coverage! 🔮 😻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Related to test code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests/std: Harness CUDA/NVCC

3 participants