[PATCH] D158259: [clang][RISCV] Support operators for RVV sizeless vector types

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/lib/Sema/SemaChecking.cpp:14906
+    // specified
+    if (S.Context.areCompatibleRVVTypes(QualType(OriginalTarget, 0),
+                                        QualType(Source, 0)) ||
----------------
Why do we need this, but SVE doesn't?


================
Comment at: clang/lib/Sema/SemaExpr.cpp:12294
 
+  if ((LHSBuiltinTy && LHSBuiltinTy->isSVEBool()) ||
+      (RHSBuiltinTy && RHSBuiltinTy->isSVEBool())) {
----------------
This code looks identical to the code above it.


================
Comment at: clang/lib/Sema/SemaExpr.cpp:16322
       break;
-    else if (resultType->isSveVLSBuiltinType()) // SVE vectors allow + and -
+    else if (resultType->isVLSBuiltinType()) // SVE vectors allow + and -
       break;
----------------
Update "SVE" to mention "RVV" too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158259/new/

https://reviews.llvm.org/D158259