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 iree_io_file_handle_create/iree_io_file_handle_open. #19510

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

benvanik
Copy link
Collaborator

@benvanik benvanik commented Dec 18, 2024

These utilities are optional but an easy way to get a file handle that is compatible with the rest of IREE. This largely replaces the
need for the existing file_io utilities but cleanup/unification is left for future changes.

Command line tools now default to --parameter_mode=file though users can force mmap or preload. There's some flags that are available on the open that may help or hurt performance and the defaults are provisional.

@benvanik benvanik added runtime/api IREE's public C runtime API and iree-run-module hal/api IREE's public C hardware abstraction layer API labels Dec 18, 2024
@benvanik benvanik force-pushed the users/benvanik/host-file branch 2 times, most recently from d1bdd7a to 5f36517 Compare December 18, 2024 05:05
@benvanik benvanik changed the base branch from main to users/benvanik/host-file-4 December 18, 2024 14:17
@benvanik benvanik changed the title [WIP] Adding support for fd-based parameter files. Adding iree_io_file_handle_create/iree_io_file_handle_open. Dec 18, 2024
@benvanik benvanik removed the hal/api IREE's public C hardware abstraction layer API label Dec 18, 2024
@benvanik benvanik marked this pull request as ready for review December 18, 2024 14:18
@benvanik benvanik force-pushed the users/benvanik/host-file branch from 5f36517 to 1dc582d 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-4 branch from 1714860 to 81efe25 Compare December 18, 2024 18:07
Base automatically changed from users/benvanik/host-file-4 to main December 18, 2024 18:27
These utilities are optional but an easy way to get a file handle
that is compatible with the rest of IREE. This largely replaces the
need for the existing file_io utilities but cleanup/unification is left
for future changes.

Command line tools now default to --parameter_mode=file though users can
force mmap or preload. There's some flags that are available on the open
that may help or hurt performance and the defaults are provisional.
@benvanik benvanik force-pushed the users/benvanik/host-file branch from 1dc582d to 7642f0c Compare December 18, 2024 20:44
@benvanik benvanik requested a review from AWoloszyn December 18, 2024 20:46
@benvanik benvanik force-pushed the users/benvanik/host-file branch from 7642f0c to fb6b582 Compare December 18, 2024 20:58
@benvanik benvanik enabled auto-merge (squash) December 18, 2024 21:25
@benvanik benvanik merged commit ce65948 into main Dec 18, 2024
42 checks passed
@benvanik benvanik deleted the users/benvanik/host-file branch December 18, 2024 21:26
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