Re: [PATCH 0/3] MR4987: ntdll: Support generic xstate (part 1).

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about dlls/ntdll/unix/system.c:
> #if defined(__i386__) || defined(__x86_64__)
>
> BOOL xstate_compaction_enabled = FALSE;
> +UINT64 xstate_supported_features_mask;
> +UINT64 xstate_features_size;
> -
>
> +unsigned int xstate_get_size( XSAVE_AREA_HEADER *xsave, UINT64 mask )
> +{
> - if (!(mask & ((UINT64)1 << XSTATE_AVX))) return sizeof(XSAVE_AREA_HEADER);
> - return sizeof(XSAVE_AREA_HEADER) + sizeof(YMMCONTEXT);
>
> +}

This helper doesn't make much sense as is, you may need to show more of part 2 to justify this.