[clang] [llvm] [RISCV] CodeGen of RVE and ilp32e/lp64e ABIs (PR #76777)
    Wang Pengcheng via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Jan 10 01:03:23 PST 2024
    
    
  
================
@@ -386,6 +393,11 @@ bool RISCVTargetInfo::handleTargetFeatures(std::vector<std::string> &Features,
   if (llvm::is_contained(Features, "+experimental"))
     HasExperimental = true;
 
+  if (ABI == "ilp32e" && ISAInfo->hasExtension("d")) {
+    Diags.Report(diag::err_invalid_feature_combination)
+        << "ILP32E cannot be used with the D ISA extension";
+    return false;
+  }
----------------
wangpc-pp wrote:
Yes, I think we should!
(Sigh...even RISC-V International has forgotten RVE, as many new ISA extensions don't even think about impacts on RVE 😞).
https://github.com/llvm/llvm-project/pull/76777
    
    
More information about the llvm-commits
mailing list