[compiler-rt] r367929 - BMI2 support is indicated in bit eight of EBX, not nine.

[Hans Wennborg via llvm-commits]

Merged to release_90 in r367996.

On Mon, Aug 5, 2019 at 11:25 PM Eric Christopher via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
>
> Author: echristo
> Date: Mon Aug  5 14:25:59 2019
> New Revision: 367929
>
> URL: http://llvm.org/viewvc/llvm-project?rev=367929&view=rev
> Log:
> BMI2 support is indicated in bit eight of EBX, not nine.
> See Intel SDM, Vol 2A, Table 3-8:
> https://www.intel.com/content/dam/www/public/us/en/documents/manuals/64-ia-32-architectures-software-developer-vol-2a-manual.pdf#page=296
>
> Differential Revision: https://reviews.llvm.org/D65766
>
> Modified:
>     compiler-rt/trunk/lib/builtins/cpu_model.c
>
> Modified: compiler-rt/trunk/lib/builtins/cpu_model.c
> URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/builtins/cpu_model.c?rev=367929&r1=367928&r2=367929&view=diff
> ==============================================================================
> --- compiler-rt/trunk/lib/builtins/cpu_model.c (original)
> +++ compiler-rt/trunk/lib/builtins/cpu_model.c Mon Aug  5 14:25:59 2019
> @@ -543,7 +543,7 @@ static void getAvailableFeatures(unsigne
>      setFeature(FEATURE_BMI);
>    if (HasLeaf7 && ((EBX >> 5) & 1) && HasAVX)
>      setFeature(FEATURE_AVX2);
> -  if (HasLeaf7 && ((EBX >> 9) & 1))
> +  if (HasLeaf7 && ((EBX >> 8) & 1))
>      setFeature(FEATURE_BMI2);
>    if (HasLeaf7 && ((EBX >> 16) & 1) && HasAVX512Save)
>      setFeature(FEATURE_AVX512F);
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits