[PATCH] D15120: Add support for __float128 type to be used by targets that support it

Hubert Tong via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 27 06:33:22 PST 2016


hubert.reinterpretcast added a comment.

In http://reviews.llvm.org/D15120#337144, @nemanjai wrote:

> Do you suggest that we need to allow operations (or at least assignments) between the two types and take away the diagnostics that are part of this patch?


My overriding concern at this time is to allow (or not) the operations consistently (that is, not making it platform-specific). The net result is that we will allow operations between the two types; however, we will encounter a further issue to discuss (@rjmccall @rsmith; your input requested) regarding what the common type should be when the representations are the same (the TS prefers the standard floating type, which is `long double` here).

As for the diagnostics, the IEEEquad to PPCDoubleDouble conversion still requires checks for narrowing.


Repository:
  rL LLVM

http://reviews.llvm.org/D15120





More information about the cfe-commits mailing list