[LoongArch][Clang] Make the parameters and return value of {x,}vorn.v builti ns unsigned char vectors (llvm#114514)

xry111 · web-flow · commit 62bf5840a63f · 2024-11-20T17:58:21.000+08:00
All other bitwise vector builtins use `unsigned char` vector operands, so it does not make too much sense to use `signed char` for the orn operation alone. The same change has been also proposed for GCC: https://gcc.gnu.org/pipermail/gcc-patches/2024-October/667065.html. Depends on llvm#114513.
diff --git a/clang/include/clang/Basic/BuiltinsLoongArchLASX.def b/clang/include/clang/Basic/BuiltinsLoongArchLASX.def
@@ -371,7 +371,7 @@ TARGET_BUILTIN(__builtin_lasx_xvor_v, "V32UcV32UcV32Uc", "nc", "lasx")
 TARGET_BUILTIN(__builtin_lasx_xvxor_v, "V32UcV32UcV32Uc", "nc", "lasx")
 TARGET_BUILTIN(__builtin_lasx_xvnor_v, "V32UcV32UcV32Uc", "nc", "lasx")
 TARGET_BUILTIN(__builtin_lasx_xvandn_v, "V32UcV32UcV32Uc", "nc", "lasx")
-TARGET_BUILTIN(__builtin_lasx_xvorn_v, "V32ScV32ScV32Sc", "nc", "lasx")
+TARGET_BUILTIN(__builtin_lasx_xvorn_v, "V32UcV32UcV32Uc", "nc", "lasx")
 
 TARGET_BUILTIN(__builtin_lasx_xvandi_b, "V32UcV32UcIUi", "nc", "lasx")
 TARGET_BUILTIN(__builtin_lasx_xvori_b, "V32UcV32UcIUi", "nc", "lasx")
diff --git a/clang/include/clang/Basic/BuiltinsLoongArchLSX.def b/clang/include/clang/Basic/BuiltinsLoongArchLSX.def
@@ -355,7 +355,7 @@ TARGET_BUILTIN(__builtin_lsx_vor_v, "V16UcV16UcV16Uc", "nc", "lsx")
 TARGET_BUILTIN(__builtin_lsx_vxor_v, "V16UcV16UcV16Uc", "nc", "lsx")
 TARGET_BUILTIN(__builtin_lsx_vnor_v, "V16UcV16UcV16Uc", "nc", "lsx")
 TARGET_BUILTIN(__builtin_lsx_vandn_v, "V16UcV16UcV16Uc", "nc", "lsx")
-TARGET_BUILTIN(__builtin_lsx_vorn_v, "V16ScV16ScV16Sc", "nc", "lsx")
+TARGET_BUILTIN(__builtin_lsx_vorn_v, "V16UcV16UcV16Uc", "nc", "lsx")
 
 TARGET_BUILTIN(__builtin_lsx_vandi_b, "V16UcV16UcIUi", "nc", "lsx")
 TARGET_BUILTIN(__builtin_lsx_vori_b, "V16UcV16UcIUi", "nc", "lsx")
diff --git a/clang/lib/Headers/lasxintrin.h b/clang/lib/Headers/lasxintrin.h
@@ -2585,7 +2585,7 @@ extern __inline
 extern __inline
     __attribute__((__gnu_inline__, __always_inline__, __artificial__)) __m256i
     __lasx_xvorn_v(__m256i _1, __m256i _2) {
-  return (__m256i)__builtin_lasx_xvorn_v((v32i8)_1, (v32i8)_2);
+  return (__m256i)__builtin_lasx_xvorn_v((v32u8)_1, (v32u8)_2);
 }
 
 #define __lasx_xvldi(/*i13*/ _1) ((__m256i)__builtin_lasx_xvldi((_1)))
diff --git a/clang/lib/Headers/lsxintrin.h b/clang/lib/Headers/lsxintrin.h
@@ -3425,7 +3425,7 @@ extern __inline
 extern __inline
     __attribute__((__gnu_inline__, __always_inline__, __artificial__)) __m128i
     __lsx_vorn_v(__m128i _1, __m128i _2) {
-  return (__m128i)__builtin_lsx_vorn_v((v16i8)_1, (v16i8)_2);
+  return (__m128i)__builtin_lsx_vorn_v((v16u8)_1, (v16u8)_2);
 }
 
 #define __lsx_vldi(/*i13*/ _1) ((__m128i)__builtin_lsx_vldi((_1)))

Original file line number	Diff line number	Diff line change
`@@ -2585,7 +2585,7 @@ extern __inline`
`2585`	`2585`	`extern __inline`
`2586`	`2586`	`__attribute__((__gnu_inline__, __always_inline__, __artificial__)) __m256i`
`2587`	`2587`	`__lasx_xvorn_v(__m256i _1, __m256i _2) {`
`2588`		`- return (__m256i)__builtin_lasx_xvorn_v((v32i8)_1, (v32i8)_2);`
	`2588`	`+ return (__m256i)__builtin_lasx_xvorn_v((v32u8)_1, (v32u8)_2);`
`2589`	`2589`	`}`
`2590`	`2590`
`2591`	`2591`	`#define __lasx_xvldi(/i13/ _1) ((__m256i)__builtin_lasx_xvldi((_1)))`
Original file line number	Diff line number	Diff line change
`@@ -3425,7 +3425,7 @@ extern __inline`
`3425`	`3425`	`extern __inline`
`3426`	`3426`	`__attribute__((__gnu_inline__, __always_inline__, __artificial__)) __m128i`
`3427`	`3427`	`__lsx_vorn_v(__m128i _1, __m128i _2) {`
`3428`		`- return (__m128i)__builtin_lsx_vorn_v((v16i8)_1, (v16i8)_2);`
	`3428`	`+ return (__m128i)__builtin_lsx_vorn_v((v16u8)_1, (v16u8)_2);`
`3429`	`3429`	`}`
`3430`	`3430`
`3431`	`3431`	`#define __lsx_vldi(/i13/ _1) ((__m128i)__builtin_lsx_vldi((_1)))`