Anastasia added a subscriber: Anastasia.
Anastasia added a comment.

Has this commit been merged without being approved by all the reviewers?


================
Comment at: cfe/trunk/lib/CodeGen/TargetInfo.cpp:5109
@@ -5108,3 +5108,3 @@
   // natively, and does not need to interwork with AAPCS code.
-  if (Ty->isHalfType() && !getContext().getLangOpts().OpenCL) {
+  if (Ty->isHalfType() && 
!getContext().getLangOpts().NativeHalfArgsAndReturns) {
     llvm::Type *ResType = IsEffectivelyAAPCS_VFP ?
----------------
Even though this change seems to be fine for OpenCL, I don't understand the 
reason for it entirely.


Repository:
  rL LLVM

http://reviews.llvm.org/D18138



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

Reply via email to