This revision was automatically updated to reflect the committed changes.
Closed by commit rL346471: [PowerPC] [Clang] [AltiVec] The second parameter of
vec_sr function should be… (authored by wuzish, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
This revision was automatically updated to reflect the committed changes.
Closed by commit rC346471: [PowerPC] [Clang] [AltiVec] The second parameter of
vec_sr function should be… (authored by wuzish, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D54087?vs=172538=173262#toc
nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.
Just for clarification (and please add the text to the commit message), this is
actually required by the ABI:
Each element of the result vector is the result of logically right shifting
wuzish created this revision.
wuzish added reviewers: hfinkel, nemanjai, kbarton.
Herald added a subscriber: jsji.
The second parameter of vec_sr function is representing shift bits and it
should be modulo the number of bits in the element like what vec_sl does now.
Repository:
rC Clang