Skip to content
This repository has been archived by the owner on Jun 20, 2019. It is now read-only.

Aarch64 vaarg #612

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Aarch64 vaarg #612

wants to merge 4 commits into from

Conversation

jpf91
Copy link
Contributor

@jpf91 jpf91 commented Feb 4, 2018

WIP preview of the AArch64 va_arg implementation. All tests pass, but this still needs some code cleanup.

This also needs patching of the backend: https://gist.github.com/jpf91/6168f248b72000d1a9010e814e120933

@ibuclaw could you take over the GDC/GCC backend part? This header include hell is driving me crazy....

@dnadlinger
Copy link

Great, something we (LDC) can just steal at last (well, the druntime bit anyway).

@ibuclaw
Copy link
Member

ibuclaw commented Apr 5, 2018

Every time I look at this, it seems like there's at least two functions that need to be now exposed per backend (all the relevant details that we care about are currently static functions) that it really irks me. Mostly because each change from static to public would need to be approved by each relevant backend maintainer. :-/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants