[PATCH] D11679: [compiler-rt] Add AArch64 to CMake configuration and several missing builtins
Sergey Dmitrouk via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 17 09:40:31 PDT 2015
sdmitrouk marked 4 inline comments as done.
sdmitrouk added a comment.
@zatrazz, thanks for the review and tests.
@rengolin, thank you for checks, review and finding reviewers!
Regards,
Sergey
================
Comment at: lib/builtins/multc3.c:21
@@ +20,3 @@
+COMPILER_RT_ABI long double _Complex
+__multc3(long double __a, long double __b, long double __c, long double __d)
+{
----------------
rengolin wrote:
> nitpick: is it necessary to have __ac and others, since they're all local context?
I was wondering the same and checked other files, some of which contain `__` prefixes, so I decided to leave as is unless it will bother anyone. I'll remove `__`.
Repository:
rL LLVM
http://reviews.llvm.org/D11679
More information about the llvm-commits
mailing list