[clang] [X86][RFC] Support AVX10 options (PR #67278)

Evgenii Kudriashov via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 17 20:23:45 PDT 2023


================
@@ -130,17 +131,35 @@ bool X86TargetInfo::initFeatureMap(
       continue;
     }
 
-    if (!HasAVX512F && Feature.substr(0, 7) == "+avx512")
+    if (Feature.substr(0, 7) == "+avx10.") {
+      HasAVX10 = true;
       HasAVX512F = true;
-    if (HasAVX512F && Feature == "-avx512f")
+      if (Feature.substr(Feature.size() - 3, 3) == "512") {
+        HasEVEX512 = true;
+      } else if (Feature.substr(7, 2) == "1-") {
----------------
e-kud wrote:

I'm not sure whether the comment here is needed or not, but I've had the first question: why don't we reset `HasEVEX512` for `avx10.2-256`. And only after realization that `avx10.2-256` implies `avx10.1-256`, it makes sense.

https://github.com/llvm/llvm-project/pull/67278


More information about the cfe-commits mailing list