================ @@ -229,7 +229,7 @@ class TargetInfo : public TransferrableTargetInfo, bool NoAsmVariants; // True if {|} are normal characters. bool HasLegalHalfType; // True if the backend supports operations on the half // LLVM IR type. - bool HalfArgsAndReturns; + bool HalfArgsAndReturns; // OpenCL 6.1.1.1, NEON (IEEE 754-2008 half) type. ---------------- JonPsson1 wrote:
I tried to make it clear that it relates to __fp16 and not _Float16, like in Type.h: ``` bool isHalfType() const; // OpenCL 6.1.1.1, NEON (IEEE 754-2008 half) bool isFloat16Type() const; // C11 extension ISO/IEC TS 18661 ``` It's all a bit confusing with all the different names for f16 types. Half seems to relate either to the LLVM I/R type, or to the ARM __fp16 type. HasLegalHalfType refers to LLVM I/R, while HalfArgsAndReturns relate to ARM (IIUC). https://github.com/llvm/llvm-project/pull/109164 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits