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

WIP: FFI Tuples #4157

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

WIP: FFI Tuples #4157

wants to merge 22 commits into from

Conversation

jemc
Copy link
Member

@jemc jemc commented Jul 7, 2022

This is a work-in-progress branch, but I'm only opening as a draft PR here so that I can begin testing all platforms via our CI jobs.

Don't bother reviewing the code right now or paying any attention to this at this time.

@jemc jemc added the do not merge This PR should not be merged at this time label Jul 7, 2022
@jemc jemc self-assigned this Jul 7, 2022
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Jul 7, 2022
@jemc jemc force-pushed the ffi-tuples branch 5 times, most recently from c4f7a43 to 7ceaddd Compare July 7, 2022 19:28
stefandd and others added 22 commits July 7, 2022 12:37
The extens the _assert_infinite() function in packages/collections/_test.pony to detect bugs in infinite Ranges
This removes the checks that were introduced in PR #2012 that only allow intrinsic FFI calls to use tuples.

Now, tuples can again be passed into and returned from FFI functions (with all previously identified limitations).
remove unused variables
Add FFI tuple compile tests to libponyc test suite
@SeanTAllen SeanTAllen removed the discuss during sync Should be discussed during an upcoming sync label Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge This PR should not be merged at this time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants