This is an archive of the discontinued LLVM Phabricator instance.

Differential D111482

[Clang] [PowerPC] Fix header include typo in smmintrin.h
ClosedPublic

Authored by qiucf on Oct 8 2021, 10:42 PM.

Details

Reviewers
jsji
nemanjai
Group Reviewers
Restricted Project
Commits
rG2fc0d439a4b6: [Clang] [PowerPC] Fix header include typo in smmintrin.h
Summary

It should be SSSE3 instead of SSE2.

Diff Detail

Event Timeline

qiucf created this revision.Oct 8 2021, 10:42 PM
Herald added subscribers: shchenz, kbarton. · View Herald TranscriptOct 8 2021, 10:42 PM
qiucf requested review of this revision.Oct 8 2021, 10:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 8 2021, 10:42 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Harbormaster completed remote builds in B127910: Diff 378414.Oct 8 2021, 11:14 PM
jsji accepted this revision as: jsji.Oct 9 2021, 1:22 PM

Can you please add a testcase that include smmintrin.h but use SSSE3? Thanks.

This revision is now accepted and ready to land.Oct 9 2021, 1:22 PM
Closed by commit rG2fc0d439a4b6: [Clang] [PowerPC] Fix header include typo in smmintrin.h (authored by qiucf). · Explain WhyOct 10 2021, 7:46 PM
This revision was automatically updated to reflect the committed changes.
qiucf added a commit: rG2fc0d439a4b6: [Clang] [PowerPC] Fix header include typo in smmintrin.h.

Revision Contents

PathSize
clang/
lib/
Headers/
ppc_wrappers/
2 lines
test/
CodeGen/
11 lines

Diff 378550

clang/lib/Headers/ppc_wrappers/smmintrin.h

Show All 26 Lines
#endif #endif
#ifndef SMMINTRIN_H_ #ifndef SMMINTRIN_H_
#define SMMINTRIN_H_ #define SMMINTRIN_H_
#if defined(__linux__) && defined(__ppc64__) #if defined(__linux__) && defined(__ppc64__)
#include <altivec.h> #include <altivec.h>
#include <emmintrin.h> #include <tmmintrin.h>
extern __inline int extern __inline int
__attribute__((__gnu_inline__, __always_inline__, __artificial__)) __attribute__((__gnu_inline__, __always_inline__, __artificial__))
_mm_extract_epi8(__m128i __X, const int __N) { _mm_extract_epi8(__m128i __X, const int __N) {
return (unsigned char)((__v16qi)__X)[__N & 15]; return (unsigned char)((__v16qi)__X)[__N & 15];
} }
extern __inline int extern __inline int
▲ Show 20 Lines • Show All 66 Lines • Show Last 20 Lines

clang/test/CodeGen/ppc-smmintrin.c

Show First 20 Lines • Show All 139 Lines • ▼ Show 20 Lines
// CHECK: %{{[0-9a-zA-Z_.]+}} = insertelement <4 x i32> %{{[0-9a-zA-Z_.]+}}, i32 %{{[0-9a-zA-Z_.]+}}, i32 %[[R0]] // CHECK: %{{[0-9a-zA-Z_.]+}} = insertelement <4 x i32> %{{[0-9a-zA-Z_.]+}}, i32 %{{[0-9a-zA-Z_.]+}}, i32 %[[R0]]
// CHECK: %[[R1:[0-9a-zA-Z_.]+]] = bitcast <4 x i32> %{{[0-9a-zA-Z_.]+}} to <2 x i64> // CHECK: %[[R1:[0-9a-zA-Z_.]+]] = bitcast <4 x i32> %{{[0-9a-zA-Z_.]+}} to <2 x i64>
// CHECK: ret <2 x i64> %[[R1]] // CHECK: ret <2 x i64> %[[R1]]
// CHECK: define available_externally <2 x i64> @_mm_insert_epi64(<2 x i64> {{[0-9a-zA-Z_%.]+}}, i64 {{[0-9a-zA-Z_%.]+}}, i32 signext {{[0-9a-zA-Z_%.]+}}) // CHECK: define available_externally <2 x i64> @_mm_insert_epi64(<2 x i64> {{[0-9a-zA-Z_%.]+}}, i64 {{[0-9a-zA-Z_%.]+}}, i32 signext {{[0-9a-zA-Z_%.]+}})
// CHECK: %[[R0:[0-9a-zA-Z_.]+]] = and i32 %{{[0-9a-zA-Z_.]+}}, 1 // CHECK: %[[R0:[0-9a-zA-Z_.]+]] = and i32 %{{[0-9a-zA-Z_.]+}}, 1
// CHECK: %{{[0-9a-zA-Z_.]+}} = insertelement <2 x i64> %{{[0-9a-zA-Z_.]+}}, i64 %{{[0-9a-zA-Z_.]+}}, i32 %[[R0:[0-9a-zA-Z_.]+]] // CHECK: %{{[0-9a-zA-Z_.]+}} = insertelement <2 x i64> %{{[0-9a-zA-Z_.]+}}, i64 %{{[0-9a-zA-Z_.]+}}, i32 %[[R0:[0-9a-zA-Z_.]+]]
// CHECK: ret <2 x i64> %{{[0-9a-zA-Z_.]+}} // CHECK: ret <2 x i64> %{{[0-9a-zA-Z_.]+}}
// To test smmintrin.h includes tmmintrin.h
void __attribute__((noinline))
test_abs_ssse3() {
_mm_abs_epi16(m1);
}
// CHECK-LABEL: @test_abs_ssse3
// CHECK: define available_externally <2 x i64> @_mm_abs_epi16(<2 x i64> {{[0-9a-zA-Z_%.]+}})