[clang] 3717147 - [clang][Sema] Only check RVV types if we have them (#67669)

[via cfe-commits]

Author: Timm Baeder
Date: 2023-10-06T09:56:05+02:00
New Revision: 3717147ee6e41f7fa3225c318104982499c5c505

URL: https://github.com/llvm/llvm-project/commit/3717147ee6e41f7fa3225c318104982499c5c505
DIFF: https://github.com/llvm/llvm-project/commit/3717147ee6e41f7fa3225c318104982499c5c505.diff

LOG: [clang][Sema] Only check RVV types if we have them (#67669)

isRVVType() is suprisingly expensive, so do the checks only if the
target has rvv types.

Added: 
    

Modified: 
    clang/lib/Sema/Sema.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/Sema/Sema.cpp b/clang/lib/Sema/Sema.cpp
index db81fa645a27ffd..67533ccbdf347c7 100644
--- a/clang/lib/Sema/Sema.cpp
+++ b/clang/lib/Sema/Sema.cpp
@@ -2071,7 +2071,7 @@ void Sema::checkTypeSupport(QualType Ty, SourceLocation Loc, ValueDecl *D) {
         targetDiag(D->getLocation(), diag::note_defined_here, FD) << D;
     }
 
-    if (Ty->isRVVType())
+    if (TI.hasRISCVVTypes() && Ty->isRVVType())
       checkRVVTypeSupport(Ty, Loc, D);
 
     // Don't allow SVE types in functions without a SVE target.