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

Move c/include/cccl/*.h files to c/include/cccl/c/*.h #2428

Merged
merged 7 commits into from
Sep 19, 2024

Conversation

rwgk
Copy link
Contributor

@rwgk rwgk commented Sep 18, 2024

Description

Closes: #2274

Piggy-backed: Change #warning to #error (to improve the user experience).

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@rwgk rwgk requested a review from a team as a code owner September 18, 2024 21:02
@rwgk rwgk requested a review from griwes September 18, 2024 21:02
Copy link
Contributor

🟩 CI finished in 14m 44s: Pass: 100%/1 | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
  • 🟩 pycuda: Pass: 100%/1 | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s

    🟩 cpu
      🟩 amd64              Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 ctk
      🟩 12.5               Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 cudacxx
      🟩 nvcc12.5           Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 cudacxx_family
      🟩 nvcc               Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 cxx
      🟩 GCC13              Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 cxx_family
      🟩 GCC                Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 gpu
      🟩 v100               Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    🟩 jobs
      🟩 Test               Pass: 100%/1   | Total: 14m 44s | Avg: 14m 44s | Max: 14m 44s
    

👃 Inspect Changes

Modifications in project?

Project
CCCL Infrastructure
libcu++
CUB
Thrust
CUDA Experimental
pycuda
+/- CUDA C Core Library

Modifications in project or dependencies?

Project
CCCL Infrastructure
libcu++
CUB
Thrust
CUDA Experimental
+/- pycuda
+/- CUDA C Core Library

🏃‍ Runner counts (total jobs: 1)

# Runner
1 linux-amd64-gpu-v100-latest-1

c/include/cccl/c/types.h Outdated Show resolved Hide resolved
c/include/cccl/c/types.h Outdated Show resolved Hide resolved
@alliepiper
Copy link
Collaborator

FYI: Merging in main to use new, smaller PR matrix.

Copy link
Contributor

🟩 CI finished in 15m 13s: Pass: 100%/1 | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
  • 🟩 pycuda: Pass: 100%/1 | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s

    🟩 cpu
      🟩 amd64              Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 ctk
      🟩 12.5               Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 cudacxx
      🟩 nvcc12.5           Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 cudacxx_family
      🟩 nvcc               Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 cxx
      🟩 GCC13              Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 cxx_family
      🟩 GCC                Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 gpu
      🟩 v100               Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    🟩 jobs
      🟩 Test               Pass: 100%/1   | Total: 15m 13s | Avg: 15m 13s | Max: 15m 13s
    

👃 Inspect Changes

Modifications in project?

Project
CCCL Infrastructure
libcu++
CUB
Thrust
CUDA Experimental
pycuda
+/- CUDA C Core Library

Modifications in project or dependencies?

Project
CCCL Infrastructure
libcu++
CUB
Thrust
CUDA Experimental
+/- pycuda
+/- CUDA C Core Library

🏃‍ Runner counts (total jobs: 1)

# Runner
1 linux-amd64-gpu-v100-latest-1

@rwgk rwgk merged commit 2fe09c8 into NVIDIA:main Sep 19, 2024
17 checks passed
@rwgk rwgk deleted the c_include_cccl_c branch September 19, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG]: Include path for C library is too generic
3 participants