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

add io_seek, operate on bytes instead of strings #386

Merged
merged 12 commits into from
Jun 18, 2024
Merged

Conversation

enricozb
Copy link
Contributor

@enricozb enricozb commented Jun 10, 2024

  • Adds a SEEK io operation.
  • replace READ_LINE and READ_CHAR with just a READ, taking in the number of bytes to be read.
  • READ and WRITE now operate on lists of bytes

@enricozb enricozb changed the title add io_seek add io_seek, operate on bytes instead of strings Jun 11, 2024
@HigherOrderBot

This comment has been minimized.

@enricozb enricozb requested a review from developedby June 11, 2024 06:01
@enricozb enricozb marked this pull request as ready for review June 11, 2024 06:01
@enricozb enricozb requested review from VictorTaelin and tjjfvi June 11, 2024 07:28
@HigherOrderBot

This comment has been minimized.

src/run.c Outdated Show resolved Hide resolved
@HigherOrderBot

This comment has been minimized.

@enricozb enricozb requested a review from developedby June 14, 2024 15:53
@HigherOrderBot

This comment has been minimized.

src/run.c Outdated Show resolved Hide resolved
@enricozb enricozb requested a review from developedby June 17, 2024 09:19
@HigherOrderBot

This comment has been minimized.

src/run.c Outdated Show resolved Hide resolved
@enricozb enricozb requested a review from developedby June 17, 2024 18:24
@HigherOrderBot
Copy link
Collaborator

Perf run for 5bbb1f5:

compiled
========

file            runtime         main            (local)       
==============================================================
sort_bitonic    c                        5.69s           3.09s
                cuda                     0.24s           0.23s
--------------------------------------------------------------
sum_rec         c                        1.36s           1.43s
                cuda                     0.15s           0.14s
--------------------------------------------------------------
sum_tree        c                        0.12s           0.11s
                cuda                     0.10s           0.10s
--------------------------------------------------------------
tuples          c                        3.80s           4.18s
                cuda                   timeout         timeout
--------------------------------------------------------------

interpreted
===========

file            runtime         main            (local)       
==============================================================
sort_bitonic    c                        4.77s           4.04s
                cuda                     0.24s           0.24s
                rust                   timeout         timeout
--------------------------------------------------------------
sum_rec         c                        1.66s           1.97s
                cuda                     0.14s           0.13s
                rust                    14.02s          14.13s
--------------------------------------------------------------
sum_tree        c                        0.39s           0.17s
                cuda                     0.08s           0.08s
                rust                     0.83s           0.84s
--------------------------------------------------------------
tuples          c                        3.69s           3.65s
                cuda                   timeout         timeout
                rust                     3.64s           3.78s
--------------------------------------------------------------


@enricozb enricozb merged commit 742a72c into main Jun 18, 2024
4 checks passed
@developedby developedby deleted the enricozb/bend-io branch June 18, 2024 09:11
@kings177 kings177 added this to the HVM IO lib v0 milestone Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants