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

Support other affinities for the STL allocator adapter #1179

Merged
merged 14 commits into from
May 2, 2024

Conversation

AnyOldName3
Copy link
Contributor

Depends on the allocator alignment changes as I've added one for the physics affinity - it's just 7f95c06 that's part of this PR, and I expect GitHub to show that as the only commit once the other work is merged.

I needed to add

        template<class U>
        struct rebind
        {
            using other = allocator_affinity_adapter<U, A>;
        };

or this wouldn't build, at least with the Microsoft STL. I'm not sure if this is required by C++ itself, or is a Microsoft STL bug, but one way or the other, it's basically the same thing as jxy-s/stlkrn#2.

@robertosfield robertosfield merged commit 81f44c5 into vsg-dev:master May 2, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants