Author: wuzish
Date: Thu Nov 8 19:35:32 2018
New Revision: 346471
URL: http://llvm.org/viewvc/llvm-project?rev=346471&view=rev
Log:
[PowerPC] [Clang] [AltiVec] The second parameter of vec_sr function should be
modulo the number of bits in the element
The second parameter of vec_sr function is r
hubert.reinterpretcast added a comment.
> some test points check the error report for ambiguous call because of too
> many implicit cast choices from ext_vector_type to vector type.
It appears the answer is to update these tests and remove the restriction on
the type class.
https://reviews.ll
wuzish added a comment.
In https://reviews.llvm.org/D53417#1292404, @rsmith wrote:
> I like the direction here, and I'd like to see this applied generally: a
> conversion sequence that bitcasts a vector should be ranked worse than one
> that does not, regardless of the kind of vector in use.
Test?
On Fri, Nov 9, 2018 at 4:34 AM Bill Wendling via cfe-commits
wrote:
>
> Author: void
> Date: Thu Nov 8 17:32:30 2018
> New Revision: 346461
>
> URL: http://llvm.org/viewvc/llvm-project?rev=346461&view=rev
> Log:
> Ignore implicit things like ConstantExpr.
>
> Modified:
>
> clang-tools
wuzish updated this revision to Diff 173276.
wuzish added a comment.
Extend the scope to all vector type as one comment suggested.
> a conversion sequence that bitcasts a vector should be ranked worse than one
> that does not, regardless of the kind of vector in use.
Which is also made code cle
wuzish marked 2 inline comments as done.
wuzish added a comment.
The updated patch now extended the scope and can include the case.
https://reviews.llvm.org/D53417
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
101 - 106 of 106 matches
Mail list logo