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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
