kamaub added a comment.
We may also need an associated test case for the changed behaviour for using
`areCompatibleVectorTypes()` instead of `areSameVectorElemTypes()`.
The test coverage should display when warnings are emitted now that we account
for vector bool, vector pixel and type qualifiers.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:9861
isLaxVectorConversion(RHSType, LHSType)) {
- if (VecType->getVectorKind() == VectorType::AltiVecVector)
+ if (VecType->getVectorKind() == VectorType::AltiVecVector ||
+ VecType->getVectorKind() == VectorType::AltiVecBool ||
----------------
Is there test coverage for these addition to the if condition?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143210/new/
https://reviews.llvm.org/D143210
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
