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 experimental iree_io_file_map_view API. #19515

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

benvanik
Copy link
Collaborator

This allows for any iree_io_file_handle_t to be mapped into host memory on platforms where doing so is supported. Eventually this will be replaced with a proper mapping object and view API that will allow us to unify the ELF loading, file_io utils, and parameter handling APIs.

@benvanik benvanik added the runtime/api IREE's public C runtime API and iree-run-module label Dec 18, 2024
@benvanik benvanik force-pushed the users/benvanik/host-file-3 branch from 2121abe to 580febb Compare December 18, 2024 14:22
@benvanik benvanik force-pushed the users/benvanik/host-file-4 branch from ef1ef57 to 1714860 Compare December 18, 2024 14:22
@benvanik benvanik force-pushed the users/benvanik/host-file-3 branch from 580febb to 76b85b0 Compare December 18, 2024 17:40
Base automatically changed from users/benvanik/host-file-3 to main December 18, 2024 18:01
Copy link
Contributor

@AWoloszyn AWoloszyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a stack of a few of the other changes as well. But for the unique changes in this PR, LGTM. (Should look a lot better once the other changes are merged and this is rebased)

This allows for any iree_io_file_handle_t to be mapped into host memory
on platforms where doing so is supported. Eventually this will be
replaced with a proper mapping object and view API that will allow us
to unify the ELF loading, file_io utils, and parameter handling APIs.
@benvanik benvanik force-pushed the users/benvanik/host-file-4 branch from 1714860 to 81efe25 Compare December 18, 2024 18:07
@benvanik benvanik enabled auto-merge (squash) December 18, 2024 18:11
@benvanik benvanik merged commit 700572c into main Dec 18, 2024
42 checks passed
@benvanik benvanik deleted the users/benvanik/host-file-4 branch December 18, 2024 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
runtime/api IREE's public C runtime API and iree-run-module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants