[LLVMdev] [PATCH] ARM64 GHC Calling Convention
Tim Northover
t.p.northover at gmail.com
Wed Nov 5 11:48:14 PST 2014
Hi Luke,
I've moved the e-mails to the llvm-commits list, which is where
patches are usually discussed.
On 4 November 2014 23:55, Luke Iannini <lukexipd at gmail.com> wrote:
> This patch adds an ARM64 calling convention for GHC to round out the
> existing set.
Thanks for working in this! It's one of the things I'd have loved to
get working in the very first AArch64 commit, but I never really knew
enough about GHC internals to do it quickly.
On the patch itself, it would be good to have some tests (in
test/CodeGen/AArch64) with some typical, trivial GHC-like functions,
to make sure everything is working as intended. Most of the changes
look reasonably sane, though only running with GHC will tell for sure.
I did have one question about the change to getThisReturnPreservedMask
though. Does GHC actually use that? I seem to remember it's a bit of
an edge-case C++ issue. If not, it would probably be best to just
assert that the calling-convention *isn't* GHC rather than giving
people reading the code something else to think about that doesn't
really matter.
Cheers.
Tim.
More information about the llvm-commits
mailing list