bruno added inline comments.

================
Comment at: lib/CodeGen/CGExprScalar.cpp:954
     if (DstTy->isFloatingPointTy()) {
-      if (!CGF.getContext().getLangOpts().HalfArgsAndReturns)
+      if (CGF.getContext().getTargetInfo().useFP16ConversionIntrinsics())
         return Builder.CreateCall(
----------------
This (and in the other places in the patch) means that regardless of 
`HalfArgsAndReturns` state we want to generate an intrinsic call if 
`useFP16ConversionIntrinsics()` is true, is that always the intended behavior?  


https://reviews.llvm.org/D40112



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

Reply via email to