amyk added inline comments.

================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:7572
+  "Current bitcast for incompatible vector types (%0 and %1) are deprecated. "
+  "The default behaviour will change to what implied by the "
+  "-fno-lax-vector-conversions option">,
----------------



================
Comment at: clang/lib/Sema/SemaOverload.cpp:1664
          !ToType->hasAttr(attr::ArmMveStrictPolymorphism))) {
+      if (!InOverloadResolution &&
+          FromType->castAs<VectorType>()->getVectorKind() ==
----------------
Just wanted to check, but is this case covered in the comment above? If it is 
not, can we add a comment for it?


================
Comment at: clang/lib/Sema/SemaOverload.cpp:13027
   // end up here.
+  //
   return SemaRef.BuildCallExpr(/*Scope*/ nullptr, NewFn.get(), LParenLoc,
----------------
Unnecessary change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126540

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to