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

Enable internal use of std::allocator related functionality #1583

Merged
merged 17 commits into from
Apr 13, 2024

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Apr 3, 2024

This ports all of the allocator related facilities to be usable internally

@miscco miscco requested review from a team as code owners April 3, 2024 07:42
@miscco miscco requested review from alliepiper and wmaxey April 3, 2024 07:42
@miscco miscco added feature request New feature or request. libcu++ For all items related to libcu++ labels Apr 3, 2024
@miscco miscco requested review from harrism and griwes April 3, 2024 07:43
@harrism
Copy link
Contributor

harrism commented Apr 4, 2024

This is massive. Perhaps the PR description should explain more? Also typo in title ("interal").

@miscco miscco changed the title Enable interal use of std::allocator related functionality Enable internal use of std::allocator related functionality Apr 4, 2024
Copy link
Collaborator

@griwes griwes left a comment

Choose a reason for hiding this comment

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

Looks good overall, the cleanup of C++11 checks in the tests is what I'm after the most by requesting changes here.

@miscco miscco force-pushed the allocator_support branch 2 times, most recently from 2581353 to a9596ed Compare April 5, 2024 09:36
@miscco miscco force-pushed the allocator_support branch 3 times, most recently from 987a9dd to 2338e42 Compare April 5, 2024 12:30
Only expose constexpr allocations when they are supported (They are not supported)
@miscco miscco requested a review from griwes April 9, 2024 07:12
@miscco
Copy link
Collaborator Author

miscco commented Apr 11, 2024

@griwes can you give this another look?

@miscco miscco enabled auto-merge (squash) April 12, 2024 12:24
@miscco miscco disabled auto-merge April 12, 2024 12:24
@miscco miscco enabled auto-merge (squash) April 12, 2024 12:24
@miscco miscco merged commit 4da34c6 into NVIDIA:main Apr 13, 2024
586 checks passed
@miscco miscco deleted the allocator_support branch April 14, 2024 08:35
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. libcu++ For all items related to libcu++
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants