This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] A new meta intrinsic header file for current and future x86 instrinsics.


> 
> If there are no further comments or objections from AMD people in the
> next 48 hours, and since this header file is intended for _Intel_
> intrinsics, it is OK for mainline.
> 
> BTW: I guess there will be a header file (ammintrin.h ?) introduced to
> include AMD defined intrinsics, so gcc can still add x86intrin.h that
> will include both immintrin.h and eventual ammintrin.h. Hopefully,
these
> two intrinsics won't step onto each other toes.
> 
> I hope this eventual solution is acceptable to both contributors.
> 
> Thanks,
> Uros.
> 

Hi,

This proposal is not acceptable to AMD for the following reasons:

- GCC is an open source compiler that is vendor agnostic. We do not want
to see vendor specific intrinsic files, as the ISAs are architecture
specific primarily. This is how it has been till now (eg: emmintrin.h,
xmmintrin.h, etc). Also there are many examples when AMD adopted SSE
instructions after Intel created them, and Intel adopted AMD
instructions after AMD came up with them (latest example: popcnt).

- We recommend that there is only 1 intrinsic file used for all x86
intrinsics. This seemed to have been agreed upon
(http://gcc.gnu.org/ml/gcc/2008-04/msg00081.html). We prefer that this
file be neutral and vendor agnostic, which immintrin.h is not.

Thanks,
Dwarak


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]