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