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

Boost.Any allocator support RFC #22

Open
apolukhin opened this issue Apr 24, 2021 · 0 comments
Open

Boost.Any allocator support RFC #22

apolukhin opened this issue Apr 24, 2021 · 0 comments

Comments

@apolukhin
Copy link
Member

Right now it is possible to use Boost.Any with types that require special alignment (sse intrinsics, Eigen3 data types, ...) by overriding the global operators new and delete. This solution is far from optimal, since it results in everything in the whole program being overaligned.

I'm opening this Request For Comments to discuss a possibly better solution to this problem.

The gist of the idea is for Boost.Any to store a polymorphic allocator (of possibly zero size), such that the user can provide an allocator when initializing/assigning/copying an any object.

This increases the API of Boost.Any (changes proposed below). It also decreases its performance, since allocating/deallocating memory would incur an extra virtual function call. It might also increase the size of the any object. One has, however, to consider the impact of the current workarounds in larger programs. Over-aligning all allocations might have a larger impact on performance and memory footprint than the proposed changes.

I think the following interface should be enough:

    any(polymorphic_allocator allocator = default_allocator()); 

    any(const any &, polymorphic_allocator allocator = default_allocator()); 

    any(any &&, polymorphic_allocator allocator = default_allocator()); 

    template<typename ValueType> any(const ValueType &, polymorphic_allocator allocator = default_allocator()); 

    template<typename ValueType> any(ValueType &&, polymorphic_allocator allocator = default_allocator()); 

Issue imported from https://svn.boost.org/trac10/ticket/11045
Reported by: gonzalobg88@…

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

No branches or pull requests

1 participant