[llvm] 2f023b9 - [RISCV] Don't make it an error have Zve* and V at the same time.

[Craig Topper via llvm-commits]

Author: Craig Topper
Date: 2022-02-01T07:16:20-08:00
New Revision: 2f023b94552ba0e53ad8cdc801eb0068c399d6da

URL: https://github.com/llvm/llvm-project/commit/2f023b94552ba0e53ad8cdc801eb0068c399d6da
DIFF: https://github.com/llvm/llvm-project/commit/2f023b94552ba0e53ad8cdc801eb0068c399d6da.diff

LOG: [RISCV] Don't make it an error have Zve* and V at the same time.

This should not be an error. V is a valid implementation of Zve.

Spec clarified here
https://github.com/riscv/riscv-v-spec/commit/9a877e8553362ff03a9b22b98e321b59aff50398

Differential Revision: https://reviews.llvm.org/D118679

Added: 
    

Modified: 
    llvm/lib/Support/RISCVISAInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Support/RISCVISAInfo.cpp b/llvm/lib/Support/RISCVISAInfo.cpp
index 8cdce750bdae..2b3395b669b8 100644
--- a/llvm/lib/Support/RISCVISAInfo.cpp
+++ b/llvm/lib/Support/RISCVISAInfo.cpp
@@ -723,12 +723,6 @@ Error RISCVISAInfo::checkDependency() {
         errc::invalid_argument,
         "zvl*b requires v or zve* extension to also be specified");
 
-  // Could not implement Zve* extension and the V extension at the same time.
-  if (HasZve32x && HasV)
-    return createStringError(
-        errc::invalid_argument,
-        "It is illegal to specify the v extension with zve* extensions");
-
   // Additional dependency checks.
   // TODO: The 'q' extension requires rv64.
   // TODO: It is illegal to specify 'e' extensions with 'f' and 'd'.