-
Notifications
You must be signed in to change notification settings - Fork 0
perf/fix: str implementation fixes (static literals, ryu, runtime cache) #74
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
Merged
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
6138ba2
docs: defer WASM target after Cranelift spike outcome
artefactop 4088621
docs: align wasm proposal with dev-doc conventions
artefactop e0dffaa
Merge remote-tracking branch 'origin' into design/milestone-8.1-spec
artefactop 4930887
feat(runtime): workspace skeleton + alloc/free/realloc
artefactop f1bcb1a
feat(runtime): from_literal, concat, eq
artefactop 3e50211
feat(runtime): int_to_str, float_to_str, bool_to_str formatters
artefactop e58882e
refactor(codegen): introduce ValueRepr enum for multi-value str
artefactop 1b4f759
refactor(codegen): cranelift_type_for audit + is_str_type guard
artefactop 688c378
feat(codegen): StrConst lowering via ryo_str_from_literal
artefactop 4214d67
feat(linker): link libryo_runtime.a into every binary
artefactop 526359e
feat(codegen): print() accepts heap-allocated str values
artefactop e1871f9
feat: str + str concatenation via ryo_str_concat
artefactop 9457310
feat: str == / != equality via ryo_str_eq
artefactop 7270ef7
feat: int_to_str, float_to_str, bool_to_str builtins
artefactop 99189ff
feat(parser): dot-method-call syntax (expr.name(args))
artefactop 2eb7b10
feat: .len() and .is_empty() method calls on str
artefactop 00a3b36
test: comprehensive str integration tests + edge cases
artefactop c78fa06
feat(codegen): str function parameters and returns via sret
artefactop ace8934
fix: runtime correctness bugs and dead code cleanup
artefactop d67ddac
docs: update roadmap ownership lattice and CLAUDE.md emphasis
artefactop 6fb04f4
perf(runtime): static string optimization — literals avoid heap alloc…
artefactop 46da72a
test(runtime): verify static/heap string interaction safety
artefactop 103b96b
fix(runtime): replace naive float_to_str with ryu for correctness
artefactop b065a59
perf(build): cache runtime archive with content hash
artefactop 82ffee5
test: integration tests for float_to_str edge cases
artefactop af4079a
style: cargo fmt
artefactop dd3a335
fix(ci): build runtime before clippy and test
artefactop 50814c9
fix: address code review findings across runtime, codegen, and sema
artefactop 51608f4
fix(linker): link libunwind on Linux to resolve _Unwind_* symbols
artefactop 903410b
docs: add I-043 (runtime no_std migration) and issue tracking convention
artefactop 957d43e
fix: pass runtime path as OsStr and validate u64→usize in str_concat
artefactop 5fa2ed4
fix: implement PR #74 review comments & auto-bootstrap runtime
artefactop eaabab1
fix: resolve clippy warnings in build.rs and format workspace
artefactop 488487c
fix(build): avoid string panics in build.rs & document profile logic
artefactop File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| [package] | ||
| name = "ryo-runtime" | ||
| version = "0.1.0" | ||
| edition = "2024" | ||
|
artefactop marked this conversation as resolved.
|
||
|
|
||
| [lib] | ||
| crate-type = ["staticlib", "rlib"] | ||
| # rlib is needed for `cargo test` to work (staticlib alone doesn't support test harness) | ||
|
|
||
| [dependencies] | ||
| ryu = "1" | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.