[PATCH] D83338: [PowerPC][Power10] Implemented Vector Shift Builtins

Albion Fung via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 12:14:09 PDT 2020


Conanap added inline comments.


================
Comment at: clang/lib/Headers/altivec.h:17243
+  // return (vector signed __int128)vec_sr((vector unsigned __int128)__a, __b);
+  return (vector signed __int128)(
+            ((vector unsigned __int128) __a) >>
----------------
amyk wrote:
> Could you please fix the indentation of the returns to make them all consistent?
I ran clang-format this time, lmk if we should change this as it doesn't look like clang-format was too consistent with itself


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83338/new/

https://reviews.llvm.org/D83338



More information about the llvm-commits mailing list