================
@@ -30,21 +30,22 @@ using namespace clang;
using namespace clang::CIRGen;
using namespace llvm;
-mlir::Value CIRGenFunction::emitAArch64SVEBuiltinExpr(unsigned builtinID,
- const CallExpr *expr) {
+std::optional<mlir::Value>
+CIRGenFunction::emitAArch64SVEBuiltinExpr(unsigned builtinID,
+ const CallExpr *expr) {
if (builtinID >= SVE::BI__builtin_sve_reinterpret_s8_s8 &&
builtinID <= SVE::BI__builtin_sve_reinterpret_f64_f64_x4) {
cgm.errorNYI(expr->getSourceRange(),
std::string("unimplemented AArch64 builtin call: ") +
getContext().BuiltinInfo.getName(builtinID));
- return {};
+ return std::nullopt;
----------------
Luhaocong wrote:
Prior to this PR, the unimplemented builtin diagnostic also were reported
twice? I think `return mlir::Value{}` under the change will avoid this issue
temporarily.
https://github.com/llvm/llvm-project/pull/171094
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
