[PATCH] D26748: [ARM] Relax restriction on variadic functions for tailcall optimization

Oliver Stannard via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 07:57:30 PST 2016


olista01 added a comment.

The change itself looks fine, I have a few suggestions for testing:

The test is checking the APCS ABI, could you add some extra RUN lines to check the newer AAPCS ABI? In particular, v_caller_ints2 will not be tail-callable in AAPCS because i64s must be in an even-odd register pair, so the last argument will go on the stack.

I'd also suggest adding some tests using doubles, and using the arm_aapcs_vfpcc calling convention, as doubles get treated differently for variadic and non-variadic calls.


https://reviews.llvm.org/D26748





More information about the llvm-commits mailing list