[clang] [RISCV] Check if v extension is enabled by the function features for the builtins not in Zve64*. (PR #112827)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 17 20:45:58 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Jim Lin (tclin914)
<details>
<summary>Changes</summary>
Fixes: https://github.com/llvm/llvm-project/issues/109694
---
Full diff: https://github.com/llvm/llvm-project/pull/112827.diff
1 Files Affected:
- (modified) clang/lib/Sema/SemaRISCV.cpp (+6-1)
``````````diff
diff --git a/clang/lib/Sema/SemaRISCV.cpp b/clang/lib/Sema/SemaRISCV.cpp
index 3da4b515b1b114..3f1c2999286f3f 100644
--- a/clang/lib/Sema/SemaRISCV.cpp
+++ b/clang/lib/Sema/SemaRISCV.cpp
@@ -623,7 +623,12 @@ bool SemaRISCV::CheckBuiltinFunctionCall(const TargetInfo &TI,
ASTContext::BuiltinVectorTypeInfo Info = Context.getBuiltinVectorTypeInfo(
TheCall->getType()->castAs<BuiltinType>());
- if (Context.getTypeSize(Info.ElementType) == 64 && !TI.hasFeature("v"))
+ const FunctionDecl *FD = SemaRef.getCurFunctionDecl();
+ llvm::StringMap<bool> FunctionFeatureMap;
+ Context.getFunctionFeatureMap(FunctionFeatureMap, FD);
+
+ if (Context.getTypeSize(Info.ElementType) == 64 && !TI.hasFeature("v")
+ && !FunctionFeatureMap.lookup("v"))
return Diag(TheCall->getBeginLoc(),
diag::err_riscv_builtin_requires_extension)
<< /* IsExtension */ true << TheCall->getSourceRange() << "v";
``````````
</details>
https://github.com/llvm/llvm-project/pull/112827
More information about the cfe-commits
mailing list