[PATCH] D49424: [PowerPC] Handle __builtin_xxpermdi the same way as GCC does

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 17 16:27:26 PDT 2018


nemanjai added inline comments.


================
Comment at: lib/CodeGen/CGBuiltin.cpp:10780
     // endian order so the shuffle mask must be adjusted for this on little
     // endian platforms (i.e. index is complemented and source vector reversed).
+    unsigned ElemIdx0 = (Index & 2) >> 1;;
----------------
efriedma wrote:
> Please fix the comment.
Ah yes. Good catch. Thanks Eli. I'll fix up the comment on the commit.


Repository:
  rC Clang

https://reviews.llvm.org/D49424





More information about the cfe-commits mailing list