Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Fix temporary allocation to actually pass the size of the allocation to allocators; introduce a caching allocator #1129

Merged
merged 1 commit into from
May 6, 2020

Conversation

brycelelbach
Copy link
Collaborator

No description provided.

@brycelelbach brycelelbach added this to the 1.9.10 milestone Apr 30, 2020
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.

Squash this and merge once CI passes.

… detail,

  because we are not yet exposing a full set of what we consider to be "useful"
  caching strategies. Will move this out into `thrust::` once we do a complete
  design.
* Also fix bugs in TLS pools and completely change `thrust::return_temporary_buffer`,
  to now accept a size and pass it through to allocators where necessary.
  `thrust::return_temporary_buffer` now ADL dispatches to three-argument (with
  size) or two-argument customizations, preferring three-argument customizations
  over two-argument customizations.
* Added tests for `(get|return)_temporary_buffer` customization.
@brycelelbach brycelelbach merged commit 16816a1 into master May 6, 2020
@brycelelbach brycelelbach deleted the feta branch May 6, 2020 18:41
@brycelelbach brycelelbach restored the feta branch May 16, 2020 06:58
@brycelelbach brycelelbach deleted the feta branch May 16, 2020 07:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants