[X86] avx512vlfp16intrin.h.h - allow _mm_cvtsh_h/_mm256_cvtsh_h to be used in constexpr (#162275)

RKSimon · web-flow · commit 21fe3d1dbb7d · 2025-10-07T12:55:05.000Z
This was missed in the earlier f16c/fp16 constexpr patches
diff --git a/clang/lib/Headers/avx512vlfp16intrin.h b/clang/lib/Headers/avx512vlfp16intrin.h
@@ -34,11 +34,13 @@
 #define __DEFAULT_FN_ATTRS128_CONSTEXPR __DEFAULT_FN_ATTRS128
 #endif
 
-static __inline__ _Float16 __DEFAULT_FN_ATTRS128 _mm_cvtsh_h(__m128h __a) {
+static __inline__ _Float16 __DEFAULT_FN_ATTRS128_CONSTEXPR
+_mm_cvtsh_h(__m128h __a) {
   return __a[0];
 }
 
-static __inline__ _Float16 __DEFAULT_FN_ATTRS256 _mm256_cvtsh_h(__m256h __a) {
+static __inline__ _Float16 __DEFAULT_FN_ATTRS256_CONSTEXPR
+_mm256_cvtsh_h(__m256h __a) {
   return __a[0];
 }
 
diff --git a/clang/test/CodeGen/X86/avx512vlfp16-builtins.c b/clang/test/CodeGen/X86/avx512vlfp16-builtins.c
@@ -17,12 +17,14 @@ _Float16 test_mm_cvtsh_h(__m128h __A) {
   // CHECK: extractelement <8 x half> %{{.*}}, i32 0
   return _mm_cvtsh_h(__A);
 }
+TEST_CONSTEXPR(_mm_cvtsh_h((__m128h){-8.0, 7.0, -6.0, 5.0, -4.0, 3.0, -2.0, 1.0}) == -8.0);
 
 _Float16 test_mm256_cvtsh_h(__m256h __A) {
   // CHECK-LABEL: test_mm256_cvtsh_h
   // CHECK: extractelement <16 x half> %{{.*}}, i32 0
   return _mm256_cvtsh_h(__A);
 }
+TEST_CONSTEXPR(_mm256_cvtsh_h((__m256h){-32.0, 31.0, -30.0, 29.0, -28.0, 27.0, -26.0, 25.0, -24.0, 23.0, -22.0, 21.0, -20.0, 19.0, -18.0, 17.0}) == -32.0);
 
 __m128h test_mm_set_sh(_Float16 __h) {
   // CHECK-LABEL: test_mm_set_sh

Original file line number	Diff line number	Diff line change
`@@ -34,11 +34,13 @@`
`34`	`34`	`#define __DEFAULT_FN_ATTRS128_CONSTEXPR __DEFAULT_FN_ATTRS128`
`35`	`35`	`#endif`
`36`	`36`
`37`		`-static __inline__ _Float16 __DEFAULT_FN_ATTRS128 _mm_cvtsh_h(__m128h __a) {`
	`37`	`+static __inline__ _Float16 __DEFAULT_FN_ATTRS128_CONSTEXPR`
	`38`	`+_mm_cvtsh_h(__m128h __a) {`
`38`	`39`	`return __a[0];`
`39`	`40`	`}`
`40`	`41`
`41`		`-static __inline__ _Float16 __DEFAULT_FN_ATTRS256 _mm256_cvtsh_h(__m256h __a) {`
	`42`	`+static __inline__ _Float16 __DEFAULT_FN_ATTRS256_CONSTEXPR`
	`43`	`+_mm256_cvtsh_h(__m256h __a) {`
`42`	`44`	`return __a[0];`
`43`	`45`	`}`
`44`	`46`
Original file line number	Diff line number	Diff line change
`@@ -17,12 +17,14 @@ _Float16 test_mm_cvtsh_h(__m128h __A) {`
`17`	`17`	`// CHECK: extractelement <8 x half> %{{.*}}, i32 0`
`18`	`18`	`return _mm_cvtsh_h(__A);`
`19`	`19`	`}`
	`20`	`+TEST_CONSTEXPR(_mm_cvtsh_h((__m128h){-8.0, 7.0, -6.0, 5.0, -4.0, 3.0, -2.0, 1.0}) == -8.0);`
`20`	`21`
`21`	`22`	`_Float16 test_mm256_cvtsh_h(__m256h __A) {`
`22`	`23`	`// CHECK-LABEL: test_mm256_cvtsh_h`
`23`	`24`	`// CHECK: extractelement <16 x half> %{{.*}}, i32 0`
`24`	`25`	`return _mm256_cvtsh_h(__A);`
`25`	`26`	`}`
	`27`	`+TEST_CONSTEXPR(_mm256_cvtsh_h((__m256h){-32.0, 31.0, -30.0, 29.0, -28.0, 27.0, -26.0, 25.0, -24.0, 23.0, -22.0, 21.0, -20.0, 19.0, -18.0, 17.0}) == -32.0);`
`26`	`28`
`27`	`29`	`__m128h test_mm_set_sh(_Float16 __h) {`
`28`	`30`	`// CHECK-LABEL: test_mm_set_sh`