efriedma 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)
----------------
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.


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

Reply via email to