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

Trampoline function call v2 #795

Merged
merged 18 commits into from
Mar 1, 2022
Merged

Trampoline function call v2 #795

merged 18 commits into from
Mar 1, 2022

Conversation

raviqqe
Copy link
Contributor

@raviqqe raviqqe commented Feb 27, 2022

Close #757.
Close #659.

@codecov-commenter
Copy link

codecov-commenter commented Feb 27, 2022

Codecov Report

Merging #795 (a1fdf57) into main (9499beb) will decrease coverage by 0.08%.
The diff coverage is 53.48%.

❗ Current head a1fdf57 differs from pull request most recent head 2042676. Consider uploading reports for the commit 2042676 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #795      +/-   ##
==========================================
- Coverage   85.29%   85.21%   -0.09%     
==========================================
  Files         316      315       -1     
  Lines       29238    29108     -130     
==========================================
- Hits        24939    24804     -135     
- Misses       4299     4304       +5     
Impacted Files Coverage Δ
lib/ffi/src/cps/error.rs 12.50% <ø> (ø)
lib/ffi/src/cps/async_stack.rs 78.70% <47.36%> (-11.30%) ⬇️
lib/ffi-macro/src/function.rs 94.69% <100.00%> (+0.14%) ⬆️
lib/hir/src/types/type_.rs 87.83% <0.00%> (-0.48%) ⬇️
lib/hir-mir/src/lib.rs 88.14% <0.00%> (-0.05%) ⬇️
lib/ffi/src/box_any.rs 0.00% <0.00%> (ø)
.../hir-mir/src/validation/try_operation_validator.rs
...b/hir-mir/src/validation/record_field_validator.rs
...r-mir/src/validation/ffi_variant_type_validator.rs
...rc/validation/duplicate_function_name_validator.rs
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9499beb...2042676. Read the comment docs.

@raviqqe raviqqe added the merge label Mar 1, 2022
@mergify mergify bot merged commit ae67b6f into main Mar 1, 2022
@mergify mergify bot deleted the feature/trampoline-v2 branch March 1, 2022 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proper tail calls in Rust FFI Stack overflow in life game example on x86_64
2 participants