ro added a comment. In https://reviews.llvm.org/D41240#955391, @rsmith wrote:
> Mechanically, the code change looks fine, but I can't comment on whether this > is a correct change for Solaris, or whether the type provided by `__float128` > would use the right floating-point representation. You will also need to > provide a test for this change. The representation of __float128 is the same on all x86 targets AFAICT (that's certainly how this is done in GCC). I've looked at the tests currently testing __float128 and except for one (to be enabled in a moment) already pass with this patch. Repository: rC Clang https://reviews.llvm.org/D41240 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
