Committed in http://llvm.org/viewvc/llvm-project?view=revision&revision=195943
Thanks, -Hao From: [email protected] [mailto:[email protected]] On Behalf Of Hao Liu Sent: Friday, November 29, 2013 10:30 AM To: [email protected] Subject: [PATCH][AArch64] Two intrinsics are expected to return float64 not return float32. Hi, This patch fixes a minor problem about arm_neon.h. 2 ALCE intrinsics have incorrect return type. For example in arm_neon.td, we define vcvt_f64: def VCVT_F64 : SInst<"vcvt_f64", "fd", "QlQUl">; But in arm_neon.h, we get following incorrect intrinsics: __ai float32x4_t vcvtq_f64_s64(int64x2_t __a) { return (float32x4_t)__builtin_neon_vcvtq_f64_v((int8x16_t)__a, 35); } __ai float32x4_t vcvtq_f64_u64(uint64x2_t __a) { return (float32x4_t)__builtin_neon_vcvtq_f64_v((int8x16_t)__a, 51); } The return type should be flaot64x2_t. There are already many tests about such intrinsics. I don't add any tests. As this is only a minor bug and I think it needs no code review, I'll commit the patch firstly. If someone has comment about this, please tell me. Thanks, -Hao
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
