[PATCH] D40673: Add _Float128 as alias to __float128 to enable compilations on Fedora27/glibc2-26

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 30 14:10:55 PST 2017


efriedma added a comment.

> _Float128 is only *sometimes* the same type as __float128.

But we don't have hppa or IA-64 backends, so we're fine for now, I think. :)



================
Comment at: lib/Frontend/InitPreprocessor.cpp:817
   DefineFloatMacros(Builder, "LDBL", &TI.getLongDoubleFormat(), "L");
+  DefineFloatMacros(Builder, "FLT128", &TI.getFloat128Format(), "Q");
+
----------------
We should have a FIXME here to switch away from the non-standard "Q" when we can.


https://reviews.llvm.org/D40673





More information about the cfe-commits mailing list