Skip to content

Commit 161cfc6

Browse files
mikolaj-pirogmikolaj-pirog
authored
[AVX10.2] Fix wrong intrinsic names after rename (llvm#126390)
In my previous PR (llvm#123656) to update the names of AVX10.2 intrinsics and mnemonics, I have erroneously deleted `_ph` from few intrinsics. This PR corrects this.
1 parent 9679735 commit 161cfc6

File tree

4 files changed

+38
-39
lines changed

4 files changed

+38
-39
lines changed

‎clang/lib/Headers/avx10_2_512convertintrin.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -213,19 +213,19 @@ _mm512_maskz_cvts2ph_hf8(__mmask64 __U, __m512h __A, __m512h __B) {
213213
(__v64qi)(__m512i)_mm512_setzero_si512());
214214
}
215215

216-
static __inline__ __m512h __DEFAULT_FN_ATTRS512 _mm512_cvthf8(__m256i __A) {
216+
static __inline__ __m512h __DEFAULT_FN_ATTRS512 _mm512_cvthf8_ph(__m256i __A) {
217217
return (__m512h)__builtin_ia32_vcvthf8_2ph512_mask(
218218
(__v32qi)__A, (__v32hf)(__m512h)_mm512_undefined_ph(), (__mmask32)-1);
219219
}
220220

221221
static __inline__ __m512h __DEFAULT_FN_ATTRS512
222-
_mm512_mask_cvthf8(__m512h __W, __mmask32 __U, __m256i __A) {
222+
_mm512_mask_cvthf8_ph(__m512h __W, __mmask32 __U, __m256i __A) {
223223
return (__m512h)__builtin_ia32_vcvthf8_2ph512_mask(
224224
(__v32qi)__A, (__v32hf)(__m512h)__W, (__mmask32)__U);
225225
}
226226

227227
static __inline__ __m512h __DEFAULT_FN_ATTRS512
228-
_mm512_maskz_cvthf8(__mmask32 __U, __m256i __A) {
228+
_mm512_maskz_cvthf8_ph(__mmask32 __U, __m256i __A) {
229229
return (__m512h)__builtin_ia32_vcvthf8_2ph512_mask(
230230
(__v32qi)__A, (__v32hf)(__m512h)_mm512_setzero_ph(), (__mmask32)__U);
231231
}

‎clang/lib/Headers/avx10_2convertintrin.h

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -381,37 +381,36 @@ _mm256_maskz_cvts2ph_hf8(__mmask32 __U, __m256h __A, __m256h __B) {
381381
(__v32qi)(__m256i)_mm256_setzero_si256());
382382
}
383383

384-
static __inline__ __m128h __DEFAULT_FN_ATTRS128 _mm_cvthf8(__m128i __A) {
384+
static __inline__ __m128h __DEFAULT_FN_ATTRS128 _mm_cvthf8_ph(__m128i __A) {
385385
return (__m128h)__builtin_ia32_vcvthf8_2ph128_mask(
386386
(__v16qi)__A, (__v8hf)(__m128h)_mm_undefined_ph(), (__mmask8)-1);
387387
}
388388

389-
static __inline__ __m128h __DEFAULT_FN_ATTRS128 _mm_mask_cvthf8(__m128h __W,
390-
__mmask8 __U,
391-
__m128i __A) {
389+
static __inline__ __m128h __DEFAULT_FN_ATTRS128
390+
_mm_mask_cvthf8_ph(__m128h __W, __mmask8 __U, __m128i __A) {
392391
return (__m128h)__builtin_ia32_vcvthf8_2ph128_mask(
393392
(__v16qi)__A, (__v8hf)(__m128h)__W, (__mmask8)__U);
394393
}
395394

396-
static __inline__ __m128h __DEFAULT_FN_ATTRS128 _mm_maskz_cvthf8(__mmask8 __U,
397-
__m128i __A) {
395+
static __inline__ __m128h __DEFAULT_FN_ATTRS128
396+
_mm_maskz_cvthf8_ph(__mmask8 __U, __m128i __A) {
398397
return (__m128h)__builtin_ia32_vcvthf8_2ph128_mask(
399398
(__v16qi)__A, (__v8hf)(__m128h)_mm_setzero_ph(), (__mmask8)__U);
400399
}
401400

402-
static __inline__ __m256h __DEFAULT_FN_ATTRS256 _mm256_cvthf8(__m128i __A) {
401+
static __inline__ __m256h __DEFAULT_FN_ATTRS256 _mm256_cvthf8_ph(__m128i __A) {
403402
return (__m256h)__builtin_ia32_vcvthf8_2ph256_mask(
404403
(__v16qi)__A, (__v16hf)(__m256h)_mm256_undefined_ph(), (__mmask16)-1);
405404
}
406405

407406
static __inline__ __m256h __DEFAULT_FN_ATTRS256
408-
_mm256_mask_cvthf8(__m256h __W, __mmask16 __U, __m128i __A) {
407+
_mm256_mask_cvthf8_ph(__m256h __W, __mmask16 __U, __m128i __A) {
409408
return (__m256h)__builtin_ia32_vcvthf8_2ph256_mask(
410409
(__v16qi)__A, (__v16hf)(__m256h)__W, (__mmask16)__U);
411410
}
412411

413412
static __inline__ __m256h __DEFAULT_FN_ATTRS256
414-
_mm256_maskz_cvthf8(__mmask16 __U, __m128i __A) {
413+
_mm256_maskz_cvthf8_ph(__mmask16 __U, __m128i __A) {
415414
return (__m256h)__builtin_ia32_vcvthf8_2ph256_mask(
416415
(__v16qi)__A, (__v16hf)(__m256h)_mm256_setzero_ph(), (__mmask16)__U);
417416
}

‎clang/test/CodeGen/X86/avx10_2_512convert-builtins.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -201,22 +201,22 @@ __m512i test_mm512_maskz_cvts2ph_hf8(__mmask64 __U, __m512h __A, __m512h __B) {
201201
return _mm512_maskz_cvts2ph_hf8(__U, __A, __B);
202202
}
203203

204-
__m512h test_mm512_cvthf8(__m256i __A) {
205-
// CHECK-LABEL: @test_mm512_cvthf8(
204+
__m512h test_mm512_cvthf8_ph(__m256i __A) {
205+
// CHECK-LABEL: @test_mm512_cvthf8_ph(
206206
// CHECK: call <32 x half> @llvm.x86.avx10.mask.vcvthf82ph512(
207-
return _mm512_cvthf8(__A);
207+
return _mm512_cvthf8_ph(__A);
208208
}
209209

210-
__m512h test_mm512_mask_cvthf8(__m512h __A, __mmask32 __B, __m256i __C) {
211-
// CHECK-LABEL: @test_mm512_mask_cvthf8(
210+
__m512h test_mm512_mask_cvthf8_ph(__m512h __A, __mmask32 __B, __m256i __C) {
211+
// CHECK-LABEL: @test_mm512_mask_cvthf8_ph(
212212
// CHECK: call <32 x half> @llvm.x86.avx10.mask.vcvthf82ph512(
213-
return _mm512_mask_cvthf8(__A, __B, __C);
213+
return _mm512_mask_cvthf8_ph(__A, __B, __C);
214214
}
215215

216-
__m512h test_mm512_maskz_cvthf8(__mmask32 __A, __m256i __B) {
217-
// CHECK-LABEL: @test_mm512_maskz_cvthf8(
216+
__m512h test_mm512_maskz_cvthf8_ph(__mmask32 __A, __m256i __B) {
217+
// CHECK-LABEL: @test_mm512_maskz_cvthf8_ph(
218218
// CHECK: call <32 x half> @llvm.x86.avx10.mask.vcvthf82ph512(
219-
return _mm512_maskz_cvthf8(__A, __B);
219+
return _mm512_maskz_cvthf8_ph(__A, __B);
220220
}
221221

222222
__m256i test_mm512_cvtph_bf8(__m512h __A) {

‎clang/test/CodeGen/X86/avx10_2convert-builtins.c

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -379,40 +379,40 @@ __m256i test_mm256_maskz_cvts2ph_hf8(__mmask16 __U, __m256h __A, __m256h __B) {
379379
return _mm256_maskz_cvts2ph_hf8(__U, __A, __B);
380380
}
381381

382-
__m128h test_mm_cvthf8(__m128i __A) {
383-
// CHECK-LABEL: @test_mm_cvthf8(
382+
__m128h test_mm_cvthf8_ph(__m128i __A) {
383+
// CHECK-LABEL: @test_mm_cvthf8_ph(
384384
// CHECK: call <8 x half> @llvm.x86.avx10.mask.vcvthf82ph128(
385-
return _mm_cvthf8(__A);
385+
return _mm_cvthf8_ph(__A);
386386
}
387387

388-
__m128h test_mm_mask_cvthf8(__m128h __A, __mmask8 __B, __m128i __C) {
389-
// CHECK-LABEL: @test_mm_mask_cvthf8(
388+
__m128h test_mm_mask_cvthf8_ph(__m128h __A, __mmask8 __B, __m128i __C) {
389+
// CHECK-LABEL: @test_mm_mask_cvthf8_ph(
390390
// CHECK: call <8 x half> @llvm.x86.avx10.mask.vcvthf82ph128(
391-
return _mm_mask_cvthf8(__A, __B, __C);
391+
return _mm_mask_cvthf8_ph(__A, __B, __C);
392392
}
393393

394-
__m128h test_mm_maskz_cvthf8(__mmask8 __A, __m128i __B) {
395-
// CHECK-LABEL: @test_mm_maskz_cvthf8(
394+
__m128h test_mm_maskz_cvthf8_ph(__mmask8 __A, __m128i __B) {
395+
// CHECK-LABEL: @test_mm_maskz_cvthf8_ph(
396396
// CHECK: call <8 x half> @llvm.x86.avx10.mask.vcvthf82ph128(
397-
return _mm_maskz_cvthf8(__A, __B);
397+
return _mm_maskz_cvthf8_ph(__A, __B);
398398
}
399399

400-
__m256h test_mm256_cvthf8(__m128i __A) {
401-
// CHECK-LABEL: @test_mm256_cvthf8(
400+
__m256h test_mm256_cvthf8_ph(__m128i __A) {
401+
// CHECK-LABEL: @test_mm256_cvthf8_ph(
402402
// CHECK: call <16 x half> @llvm.x86.avx10.mask.vcvthf82ph256(
403-
return _mm256_cvthf8(__A);
403+
return _mm256_cvthf8_ph(__A);
404404
}
405405

406-
__m256h test_mm256_mask_cvthf8(__m256h __A, __mmask16 __B, __m128i __C) {
407-
// CHECK-LABEL: @test_mm256_mask_cvthf8(
406+
__m256h test_mm256_mask_cvthf8_ph(__m256h __A, __mmask16 __B, __m128i __C) {
407+
// CHECK-LABEL: @test_mm256_mask_cvthf8_ph(
408408
// CHECK: call <16 x half> @llvm.x86.avx10.mask.vcvthf82ph256(
409-
return _mm256_mask_cvthf8(__A, __B, __C);
409+
return _mm256_mask_cvthf8_ph(__A, __B, __C);
410410
}
411411

412-
__m256h test_mm256_maskz_cvthf8(__mmask16 __A, __m128i __B) {
413-
// CHECK-LABEL: @test_mm256_maskz_cvthf8(
412+
__m256h test_mm256_maskz_cvthf8_ph(__mmask16 __A, __m128i __B) {
413+
// CHECK-LABEL: @test_mm256_maskz_cvthf8_ph(
414414
// CHECK: call <16 x half> @llvm.x86.avx10.mask.vcvthf82ph256(
415-
return _mm256_maskz_cvthf8(__A, __B);
415+
return _mm256_maskz_cvthf8_ph(__A, __B);
416416
}
417417

418418
__m128i test_mm_cvtph_bf8(__m128h __A) {

0 commit comments

Comments
 (0)