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] begin implementing "ABI hooks" #1264

Closed
wants to merge 1 commit into from
Closed

[WIP] begin implementing "ABI hooks" #1264

wants to merge 1 commit into from

Conversation

eduardosm
Copy link
Contributor

This is WIP and rather useless and broken right now. I'm having issues while walking through param_info in FnTypeId. It looks like there are parameters with type->type_ref set to null, which I don't know why.

@andrewrk
Copy link
Member

I wonder, would it be more valuable to start this work in the self-hosted compiler? It's not very far along yet, but it's progressing rapidly. But once it's done we don't really need this feature in stage1.

@eduardosm
Copy link
Contributor Author

I believe you are right, I'll take a look at the self-hosted compiler.

@andrewrk
Copy link
Member

Sorry about that, I perhaps should have made you aware of that earlier.

If you look into it, I recommend looking at my branch, self-hosted-libc-hello-world. I'm trying to get this merged into master as soon as possible - just need to do a little more windows and macos work.

@andrewrk
Copy link
Member

(the above comment is now outdated as the branch has been merged into master)

@andrewrk andrewrk added the work in progress This pull request is not ready for review yet. label Aug 20, 2018
@andrewrk andrewrk added this to the 0.4.0 milestone Aug 25, 2018
@andrewrk andrewrk closed this in 7505529 Sep 7, 2018
@andrewrk andrewrk modified the milestones: 0.4.0, 0.3.0 Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work in progress This pull request is not ready for review yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants