[PATCH] D143210: [PowerPC] Include vector bool and pixel when emitting lax warning

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 qualifier

[PATCH] D143210: [PowerPC] Include vector bool and pixel when emitting lax warning

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG02a71b05fc67: [PowerPC] Include vector bool and pixel when emitting lax warning (authored by maryammo). Changed prior to

[PATCH] D143210: [PowerPC] Include vector bool and pixel when emitting lax warning

maryammo added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:9845 if (anyAltivecTypes(RHSType, LHSType) && -!areSameVectorElemTypes(RHSType, LHSType)) +!Context.areCompatibleVectorTypes(RHSType, LHSType)) Diag(RHS.get()->getE

[PATCH] D143210: PowerPC] Include vector bool and pixel when emitting lax warning

amyk added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:9845 if (anyAltivecTypes(RHSType, LHSType) && -!areSameVectorElemTypes(RHSType, LHSType)) +!Context.areCompatibleVectorTypes(RHSType, LHSType)) Diag(RHS.get()->getExprL

[PATCH] D143210: PowerPC] Include vector bool and pixel when emitting lax warning

maryammo created this revision. Herald added subscribers: shchenz, nemanjai. Herald added a project: All. maryammo requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch is to fix some missing lax-vector-conversion warnings including ca