[llvm] 94e7d9c - [RISCV] Remove Zvk* dependency checks from RISCVISAInfo::checkDependency.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 13:57:30 PDT 2024


Author: Craig Topper
Date: 2024-10-29T13:57:23-07:00
New Revision: 94e7d9c0bfe517507ea08b00fb00c32fb2837a82

URL: https://github.com/llvm/llvm-project/commit/94e7d9c0bfe517507ea08b00fb00c32fb2837a82
DIFF: https://github.com/llvm/llvm-project/commit/94e7d9c0bfe517507ea08b00fb00c32fb2837a82.diff

LOG: [RISCV] Remove Zvk* dependency checks from RISCVISAInfo::checkDependency.

The Zvk* extensions now imply Zve32x or Zve64x so it shouldn't be
possible to fail these dependency checks.

Added: 
    

Modified: 
    llvm/lib/TargetParser/RISCVISAInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/TargetParser/RISCVISAInfo.cpp b/llvm/lib/TargetParser/RISCVISAInfo.cpp
index caa5a97747ee57..de5b5c39c9ed27 100644
--- a/llvm/lib/TargetParser/RISCVISAInfo.cpp
+++ b/llvm/lib/TargetParser/RISCVISAInfo.cpp
@@ -751,17 +751,6 @@ Error RISCVISAInfo::checkDependency() {
   if (HasZvl && !HasVector)
     return getExtensionRequiresError("zvl*b", "v' or 'zve*");
 
-  if (!HasVector)
-    for (auto Ext :
-         {"zvbb", "zvbc32e", "zvkb", "zvkg", "zvkgs", "zvkned", "zvknha", "zvksed", "zvksh"})
-      if (Exts.count(Ext))
-        return getExtensionRequiresError(Ext, "v' or 'zve*");
-
-  if (!Exts.count("zve64x"))
-    for (auto Ext : {"zvknhb", "zvbc"})
-      if (Exts.count(Ext))
-        return getExtensionRequiresError(Ext, "v' or 'zve64*");
-
   if ((HasZcmt || Exts.count("zcmp")) && HasD && (HasC || Exts.count("zcd")))
     return getError(Twine("'") + (HasZcmt ? "zcmt" : "zcmp") +
                     "' extension is incompatible with '" +


        


More information about the llvm-commits mailing list