bruno added inline comments.
================
Comment at: lib/Basic/TargetInfo.cpp:229
switch (BitWidth) {
- case 96:
+ case 80:
if (&getLongDoubleFormat() == &llvm::APFloat::x87DoubleExtended)
----------------
The change makes sense but I believe there's some historical reason why this is
96 instead of 80? What problem have you found in practice? Do you have a
testcase or unittest that exercise the issue (and that could be added to the
patch)?
https://reviews.llvm.org/D26955
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
