Skip to content

Conversation

@phoebewang
Copy link
Contributor

They are defined in mmintrin.h

They are defined in mmintrin.h
@phoebewang phoebewang requested a review from RKSimon August 15, 2025 15:35
@llvmbot llvmbot added clang Clang issues not falling into any other category backend:X86 clang:headers Headers provided by Clang, e.g. for intrinsics labels Aug 15, 2025
@llvmbot
Copy link
Member

llvmbot commented Aug 15, 2025

@llvm/pr-subscribers-backend-x86

@llvm/pr-subscribers-clang

Author: Phoebe Wang (phoebewang)

Changes

They are defined in mmintrin.h


Full diff: https://github.com/llvm/llvm-project/pull/153820.diff

2 Files Affected:

  • (modified) clang/lib/Headers/emmintrin.h (-4)
  • (modified) clang/lib/Headers/xmmintrin.h (-1)
diff --git a/clang/lib/Headers/emmintrin.h b/clang/lib/Headers/emmintrin.h index 45f2d157e5c7f..38dd462e650e2 100644 --- a/clang/lib/Headers/emmintrin.h +++ b/clang/lib/Headers/emmintrin.h @@ -17,7 +17,6 @@ #include <xmmintrin.h> typedef double __m128d __attribute__((__vector_size__(16), __aligned__(16))); -typedef long long __m128i __attribute__((__vector_size__(16), __aligned__(16))); typedef double __m128d_u __attribute__((__vector_size__(16), __aligned__(1))); typedef long long __m128i_u @@ -25,9 +24,6 @@ typedef long long __m128i_u /* Type defines. */ typedef double __v2df __attribute__((__vector_size__(16))); -typedef long long __v2di __attribute__((__vector_size__(16))); -typedef short __v8hi __attribute__((__vector_size__(16))); -typedef char __v16qi __attribute__((__vector_size__(16))); /* Unsigned types */ typedef unsigned long long __v2du __attribute__((__vector_size__(16))); diff --git a/clang/lib/Headers/xmmintrin.h b/clang/lib/Headers/xmmintrin.h index e3c280b1c39ec..6d44cff46661f 100644 --- a/clang/lib/Headers/xmmintrin.h +++ b/clang/lib/Headers/xmmintrin.h @@ -16,7 +16,6 @@ #include <mmintrin.h> -typedef int __v4si __attribute__((__vector_size__(16))); typedef float __v4sf __attribute__((__vector_size__(16))); typedef float __m128 __attribute__((__vector_size__(16), __aligned__(16))); 
@phoebewang phoebewang merged commit b0d2b57 into llvm:main Aug 15, 2025
13 checks passed
@phoebewang phoebewang deleted the remove__v8hu branch August 15, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend:X86 clang:headers Headers provided by Clang, e.g. for intrinsics clang Clang issues not falling into any other category

3 participants