[PATCH] D21678: Fix For pr28288 - Error message in shift of vector values

[Akira Hatanaka via cfe-commits]

I looked at r278501 and realized it’s doing more than just fixing the test case in PR28288. For example, when the following code was compiled,

vi8 = vi8 << vuc8; // vuc8: <8 x unsigned char>, vi8:  <8 x int>
vc8 = vc8 << vs4; // vc8: <8 x char>,  vs4: <4 x short>

prior to r278501, clang would reject the first statement but accept the second, while after r278501, it would accept the first and reject the second. gcc 6.2 rejects both.

Was this change discussed thoroughly and is this really the behavior we want?

> On Sep 7, 2016, at 12:53 AM, Vladimir Yakovlev <vbyakovl.1 at gmail.com> wrote:
> 
> I did another fix (attached). It fixes the error and chang-check has not any new errors. Please take a look.
> 
> Where can I see your fix?
> 
> ​
> <patch.p>