[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