efriedma added a comment.

Probably should have a test for something like `float x = (__uint128_t)-1;`, to 
make sure we print something reasonable.



================
Comment at: lib/Sema/SemaChecking.cpp:10874
+      if (Target->isSpecificBuiltinType(BuiltinType::LongDouble))
+        FloatSem = &llvm::APFloat::x87DoubleExtended();
+
----------------
ASTContext::getFloatTypeSemantics.  (Your explicit computation here is both 
redundant and wrong.)


https://reviews.llvm.org/D52835



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

Reply via email to