[PATCH] D54087: [PowerPC] [Clang] [AltiVec] The second parameter of vec_sr function should be modulo the number of bits in the element

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 8 08:26:34 PST 2018


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 the corresponding
  element of ARG1 by the number of bits specified by the value of the corresponding
  element of ARG2, modulo the number of bits in the element. The bits that are shifted out
  are replaced by zeros.


Repository:
  rC Clang

https://reviews.llvm.org/D54087





More information about the cfe-commits mailing list