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

Mostly flatten the folder structure of libcu++ #1630

Merged
merged 5 commits into from
Apr 18, 2024

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Apr 15, 2024

This moves all of the subfolders from cuda/std/detail/libcxx/include/__meow down to cuda/std/__meow

We also move the ptx headers to cuda/__ptx and adopt the __cccl_config

This is a first step in completely flatting our libcu++ folder structure.

The remaining work is in moving all the individual header files around.

@miscco miscco requested review from a team as code owners April 15, 2024 09:20
@miscco miscco added feature request New feature or request. libcu++ For all items related to libcu++ infrastructure Shared CMake, github, etc infrastructure labels Apr 15, 2024
@miscco miscco force-pushed the move_libcudacxx branch 2 times, most recently from 4d6eb01 to 97a73bf Compare April 17, 2024 06:43
miscco added 5 commits April 18, 2024 08:33
We want to name the folder cotnaining the subheaders as `__meow` so that we then can easily expose the internal header without any further code movement
We do not need the deep nested structure anymore, as a first step move all internal subfolders of `cuda/std/detail/libcxx/include` to `cuda/std`
@wmaxey wmaxey merged commit 29a24f4 into NVIDIA:main Apr 18, 2024
587 checks passed
@miscco miscco deleted the move_libcudacxx branch June 12, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request. infrastructure Shared CMake, github, etc infrastructure libcu++ For all items related to libcu++
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants