-
Notifications
You must be signed in to change notification settings - Fork 105
Open
Labels
P1: Should haveNecessary but not criticalNecessary but not criticalgood first issueGood for newcomersGood for newcomerstopic: buildCMake build issueCMake build issuetype: feature requestNew feature requestNew feature request
Description
Is your feature request related to a problem? Please describe.
After @PointKernel's excellent work in #141 to ensure each cuco header is standalone, we should add some tests to ensure they stay that way.
Describe the solution you'd like
@allisonvacanti pointed out the fact that Thrust does this by simply including each header in an otherwise empty .cu file and ensuring it compiles. We should steal what they do.
See
CMake: https://github.com/NVIDIA/thrust/blob/main/cmake/ThrustHeaderTesting.cmake
Configured header: https://github.com/NVIDIA/thrust/blob/main/cmake/header_test.in
Metadata
Metadata
Assignees
Labels
P1: Should haveNecessary but not criticalNecessary but not criticalgood first issueGood for newcomersGood for newcomerstopic: buildCMake build issueCMake build issuetype: feature requestNew feature requestNew feature request