[PATCH] D112975: Fix complex types declared using mode TC

Elizabeth Andrews via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 2 09:14:47 PDT 2021


eandrews added a comment.

In D112975#3101720 <https://reviews.llvm.org/D112975#3101720>, @rjmccall wrote:

> For posterity in case someone tracks down this review: `TC` corresponds to an unspecified 128-bit format, which on some targets is a double-double format (like `__ibm128_t`) and on others is `float128_t`.  The bug in the previous patch is that  `long double` is only safe to use when it's known to be one of those formats.
>
> Patch LGTM.

Thanks for the note. I'll update the commit message when I check this in.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112975/new/

https://reviews.llvm.org/D112975



More information about the cfe-commits mailing list