D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 16398 - experimental allocators, add aligned reallocation for Posix
Summary: experimental allocators, add aligned reallocation for Posix
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: All All
: P1 enhancement
Assignee: No Owner
URL:
Keywords: pull
Depends on:
Blocks:
 
Reported: 2016-08-17 12:54 UTC by Илья Ярошенко
Modified: 2020-03-21 03:56 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Илья Ярошенко 2016-08-17 12:54:42 UTC
std.experimental.allocator.mallocator. AlignedMallocator does not have alignedRealocate
Comment 1 basile-z 2017-11-13 00:01:36 UTC
This is an ER only for Posix actually. Under Windows, 
1. the MS std C lib has the right primitive.
2. the allocator has custom primitives for the Mars std C lib (which does a new alloc + copy)

Under Posix, the alligned alloc routines exist but only alloc / free.
So because of 2. i don't see why alloc + copy wouldn't not be done for Posix.
Comment 2 basile-z 2017-11-13 02:01:21 UTC
PR https://github.com/dlang/phobos/pull/5857
Comment 3 github-bugzilla 2017-11-13 21:22:39 UTC
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a196f171f6cc0d6936d7b2d5fc4fc7e38b026514
fix issue 16398 - experimental allocators, add aligned reallocation for Posix

https://github.com/dlang/phobos/commit/0308bd6b9a080501b931f43d1e39b1c3cf79c29a
Merge pull request #5857 from BBasile/issue-16398

fix issue 16398 - experimental allocators, add aligned reallocation for Posix
Comment 4 github-bugzilla 2017-12-18 22:56:47 UTC
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a196f171f6cc0d6936d7b2d5fc4fc7e38b026514
fix issue 16398 - experimental allocators, add aligned reallocation for Posix

https://github.com/dlang/phobos/commit/0308bd6b9a080501b931f43d1e39b1c3cf79c29a
Merge pull request #5857 from BBasile/issue-16398