Re: PATCH: Enable Intel AES/CLMUL

["Uros Bizjak" <ubizjak@gmail.com>]

On Fri, Apr 4, 2008 at 3:28 PM, H.J. Lu <hjl.tools@gmail.com> wrote:

>  > >  SSE doesn't support V2DI. You need at least SSE2.
>  > >
>  >
>  > OK.

>  > This practically forces usage of -msse4.1 for -maes/-mclmul, since
>  > these instructions are used only through intrinsics, defined in their
>  > corresponding header file. wmmintrin.h includes smmintrin.h that will
>  > error out by #ifndef __SSE4_1__.
>  >
>  > I think that user should explicitly enable sse4.1 manually in the
>  > command line together with -maes/-mclmul to access these new
>  > instructions. Including wmmintrin.h will error out when neither
>  > -maes/-mclmul is enabled, and smmintrin.h will error out with a
>  > message that SSE4.1 instruction set is not enabled.
>
>  There is a proposal to get rid of all *mmintrin.h.  Users should
>  include one header file, something like <ia32intrin.h> ,for all
>  current and future intrinsics.  The name of the meta intrinsic
>  heade file hasn't be decided. Do we have any preferences/suggestions?
>
>  With that in mind, I will change it to include emmintrin.h.

OK. So, we require user to pass at least -msse2 in addition to
-maes/-mclmul then? The errors will be informative, so there will be
no confusion.

>  > "instruction set" or "instructions" in the error message?
>
>  I can use instructions.

This is up to your taste, "instruction set" just sound too fancy for
me for a pack of instructions ;)

Uros.