[PATCH] D82359: [Power10] Implement Vector Replace Builtins in LLVM/Clang

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added inline comments.


================
Comment at: clang/lib/Headers/altivec.h:16839
+      (__b), signed int                                                        \
+      : (vector signed int)__builtin_altivec_vinsw((signed int)(__b),          \
+                                                   (12 - ((__c)*4))),          \
----------------
I see that the value range for __c is 0, 3. Could you add some semantic check in frontend about this?


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

https://reviews.llvm.org/D82359