[PATCH] D18138: Add -fnative-half-arguments-and-returns

Anastasia Stulova via cfe-commits cfe-commits at lists.llvm.org
Fri May 20 12:41:28 PDT 2016


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





More information about the cfe-commits mailing list