Skip to content

Commit 9442b4e

Browse files
committed
[X86] mmx-builtins.c - use __v8qs initializer instead of _mm_setr_pi8 to correctly run on -fno-signed-char targets
1 parent bbde6be commit 9442b4e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

clang/test/CodeGen/X86/mmx-builtins.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ __m64 test_mm_add_pi8(__m64 a, __m64 b) {
3434
// CHECK: add <8 x i8> {{%.*}}, {{%.*}}
3535
return _mm_add_pi8(a, b);
3636
}
37-
TEST_CONSTEXPR(match_v8qi(_mm_add_pi8(_mm_setr_pi8(-3, +2, -1, 0, +1, -2, +3, -4), _mm_setr_pi8(-18, +16, -14, +12, -10, +8, +6, -4)), -21, +18, -15, +12, -9, +6, +9, -8));
37+
TEST_CONSTEXPR(match_v8qi(_mm_add_pi8((__m64)(__v8qs){-3, +2, -1, 0, +1, -2, +3, -4}, (__m64)(__v8qs){-18, +16, -14, +12, -10, +8, +6, -4}), -21, +18, -15, +12, -9, +6, +9, -8));
3838

3939
__m64 test_mm_add_pi16(__m64 a, __m64 b) {
4040
// CHECK-LABEL: test_mm_add_pi16
@@ -120,7 +120,7 @@ __m64 test_mm_cmpeq_pi8(__m64 a, __m64 b) {
120120
// CHECK-NEXT: {{%.*}} = sext <8 x i1> [[CMP]] to <8 x i8>
121121
return _mm_cmpeq_pi8(a, b);
122122
}
123-
TEST_CONSTEXPR(match_v8qi(_mm_cmpeq_pi8(_mm_setr_pi8(-3, +2, -1, 0, +1, -2, +3, -4), _mm_setr_pi8(-3, -2, +1, 0, -1, -2, -3, -4)), -1, 0, 0, -1, 0, -1, 0, -1));
123+
TEST_CONSTEXPR(match_v8qi(_mm_cmpeq_pi8((__m64)(__v8qs){-3, +2, -1, 0, +1, -2, +3, -4}, (__m64)(__v8qs){-3, -2, +1, 0, -1, -2, -3, -4}), -1, 0, 0, -1, 0, -1, 0, -1));
124124

125125
__m64 test_mm_cmpeq_pi16(__m64 a, __m64 b) {
126126
// CHECK-LABEL: test_mm_cmpeq_pi16
@@ -144,7 +144,7 @@ __m64 test_mm_cmpgt_pi8(__m64 a, __m64 b) {
144144
// CHECK-NEXT: {{%.*}} = sext <8 x i1> [[CMP]] to <8 x i8>
145145
return _mm_cmpgt_pi8(a, b);
146146
}
147-
TEST_CONSTEXPR(match_v8qi(_mm_cmpgt_pi8(_mm_setr_pi8(-3, +2, -1, 0, +1, -2, +3, -4), _mm_setr_pi8(-3, -2, +1, 0, -1, -2, -3, -4)), 0, -1, 0, 0, -1, 0, -1, 0));
147+
TEST_CONSTEXPR(match_v8qi(_mm_cmpgt_pi8((__m64)(__v8qs){-3, +2, -1, 0, +1, -2, +3, -4}, (__m64)(__v8qs){-3, -2, +1, 0, -1, -2, -3, -4}), 0, -1, 0, 0, -1, 0, -1, 0));
148148

149149
__m64 test_mm_cmpgt_pi16(__m64 a, __m64 b) {
150150
// CHECK-LABEL: test_mm_cmpgt_pi16
@@ -690,7 +690,7 @@ __m64 test_mm_sub_pi8(__m64 a, __m64 b) {
690690
// CHECK: sub <8 x i8> {{%.*}}, {{%.*}}
691691
return _mm_sub_pi8(a, b);
692692
}
693-
TEST_CONSTEXPR(match_v8qi(_mm_sub_pi8(_mm_setr_pi8(-3, +2, -1, 0, +1, -2, +3, -4), _mm_setr_pi8(-18, +16, -14, +12, -10, +8, +6, -4)), +15, -14, +13, -12, +11, -10, -3, 0));
693+
TEST_CONSTEXPR(match_v8qi(_mm_sub_pi8((__m64)(__v8qs){-3, +2, -1, 0, +1, -2, +3, -4}, (__m64)(__v8qs){-18, +16, -14, +12, -10, +8, +6, -4}), +15, -14, +13, -12, +11, -10, -3, 0));
694694

695695
__m64 test_mm_sub_pi16(__m64 a, __m64 b) {
696696
// CHECK-LABEL: test_mm_sub_pi16

0 commit comments

Comments
 (0)