[PATCH] D18769: Swift Calling Convention: swiftcc implementation on ARM
Manman Ren via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 4 12:28:34 PDT 2016
manmanren added a comment.
In http://reviews.llvm.org/D17863#371104, @t.p.northover wrote:
> Hi Manman,
>
> I think the testing is too weak if we're going for ABI stability (particularly based on FastCC, which is explicitly called out in the LangRef as unstable and based on APCS). I'd actually strongly suggest Swift starts out with AAPCS_VFP on ARM.
This is now updated to use AAPCS_VFP.
> Even with that, I'd suggest testing things like minimum stack slot size, alignment of objects on the stack, whether everything goes on the stack if something has. Probably on both iOS and WatchOS because I suspect those details differ based on type alignment.
Do the testings for WatchOS already exist? If yes, can you point me which file so it will be easier for me to add coverage for iOS?
For Swift on WatchOS, we are using the same convention as C/C++, so I don't quite get what extra testings we would like to include for Swift.
Thanks,
Manman
http://reviews.llvm.org/D18769
More information about the llvm-commits
mailing list