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

Adding split iree_hal_buffer_prepare_map_range/commit_map_range. #18159

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

benvanik
Copy link
Collaborator

@benvanik benvanik commented Aug 8, 2024

This allows for the validation/offsetting/etc that acts only on metadata to occur separate from the actual commit of the mapping where the host storage is accessed. Code that wants to validate early (such as during command buffer recording) can now do so on uncommitted buffers and have a fast-path for when the underlying storage is available.

This will be used for native implementations of reusable command buffers by allowing them to validate direct and indirect bindings during recording even if the bindings are of uncommitted buffers (#18158) that won't be available until execution-time.

This allows for the validation/offsetting/etc that acts only on metadata
to occur separate from the actual commit of the mapping where the
host storage is accessed. Code that wants to validate early (such as
during command buffer recording) can now do so on uncommitted buffers
and have a fast-path for when the underlying storage is available.
@benvanik benvanik added the hal/api IREE's public C hardware abstraction layer API label Aug 8, 2024
@benvanik benvanik requested a review from ScottTodd August 8, 2024 18:24
@benvanik benvanik marked this pull request as ready for review August 8, 2024 18:53
@benvanik benvanik enabled auto-merge (squash) August 8, 2024 18:53
@benvanik benvanik merged commit fbf677d into main Aug 8, 2024
49 checks passed
@benvanik benvanik deleted the users/benvanik/split-map-range branch August 8, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hal/api IREE's public C hardware abstraction layer API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants