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