[PATCH] D53608: [builtins] Build float128 soft float builtins for x86_64.

Manoj Gupta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 24 13:35:08 PDT 2018


manojgupta added a comment.

Took another look and seems like long double is hardcoded in many of the builtins. So I think the current patch needs to rename a lot of places using long double to __float128 type.

Some examples where I think __float128 type (propagating the type in fp_lib.h) should be used instead of long double:

./extenddftf2.c:  COMPILER_RT_ABI long double __extenddftf2(double a) {
./trunctfsf2.c:  COMPILER_RT_ABI float __trunctfsf2(long double a) {
./extendsftf2.c:  COMPILER_RT_ABI long double __extendsftf2(float a) {

@efriedma what do you think?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D53608





More information about the cfe-commits mailing list