Skip to content

Implement cpython support in test_c_interop_03 - #603

Merged
certik merged 2 commits into
lcompilers:mainfrom
certik:cp2
Jun 14, 2022
Merged

certik merged 2 commits into
lcompilers:mainfrom
certik:cp2

Conversation

@certik

@certik certik commented Jun 14, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@certik

certik commented Jun 14, 2022

Copy link
Copy Markdown
Contributor Author

The test passes, but the expression pointer(xi32, i32) returns a pointer to a copy of the integer xi32, as Python passes a new copy of the integer (as value), so this approach does not allow us to take a pointer to it. I think it's probably fine, we just can't use it for return values. For NumPy arrays arguments it will return a pointer to the actual data.

@certik certik mentioned this pull request Jun 14, 2022
7 of 11 tasks
@certik
certik enabled auto-merge June 14, 2022 13:27
@certik

certik commented Jun 14, 2022

Copy link
Copy Markdown
Contributor Author

This gets it done, in a "prototype" mode. As we get more experience actually using this, we'll "harden" it.

@certik
certik merged commit 8745d6d into lcompilers:main Jun 14, 2022
@certik
certik deleted the cp2 branch June 14, 2022 13:40
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.

1 participant