[PATCH] D158811: [X86] __builtin_cpu_supports: support x86-64{,-v2,-v3,-v4}

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/TargetParser/X86TargetParser.cpp:718
         ;
-    FeaturesMask |= (1ULL << Feature);
+    FeatureMask[Feature / 32] |= 1U << (Feature % 32);
   }
----------------
pengfei wrote:
> Should we use vector for future expansion, or add an assert to make sure it won't exceed current limitation?
A resizable vector does not simplify code here. I've added an assert.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158811/new/

https://reviews.llvm.org/D158811