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