manojgupta added inline comments.
================ Comment at: lib/builtins/fp_lib.h:111 typedef __int128_t srep_t; -typedef long double fp_t; +typedef __float128 fp_t; #define REP_C (__uint128_t) ---------------- efriedma wrote: > manojgupta wrote: > > Changed long double to __float128 on Eli's advice in PR39376. > You need to guard this with an ifdef; clang doesn't supports __float128 on > every target, even if long double is an 128-bit IEEE float. > > For reasons I don't really understand, there are apparently two different > macros for this; `#if defined(__FLOAT128__) || defined(__SIZEOF_FLOAT128__)` > should do the right thing. Thanks Eli. I also found out that GCC 4.9 does not seem to have these defined even though it supports __float128. https://godbolt.org/z/ReVm8j Repository: rCRT Compiler Runtime https://reviews.llvm.org/D53608 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
