xbolva00 added inline comments.

================
Comment at: lib/Sema/SemaChecking.cpp:10862
+  if (Source->isIntegerType() && Target->isFloatingType()) {
+    const llvm::fltSemantics *FloatSem = nullptr;
+    if (Target->isSpecificBuiltinType(BuiltinType::Float)) {
----------------
erichkeane wrote:
> Since there is only 1, and fltSemantics is really small (2 shorts + 2 uints), 
> making this a 'by copy' is likely a better solution than a pointer.
llvm::fltSemantics has incomplete type and cannot be defined :/ Quite strange 
error


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