rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.

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.


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

https://reviews.llvm.org/D112975

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to