[PATCH] D89446: [WebAssembly] Prototype i8x16.popcnt

[Heejin Ahn via Phabricator via llvm-commits]

aheejin accepted this revision.
aheejin added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:117
 
+TARGET_BUILTIN(__builtin_wasm_popcnt, "V16ScV16Sc", "nc", "simd128")
+
----------------
- Even if there's only one vector type for a builtin, it seems others still have type postfix attached.
- Is the result also a vector? Does that mean this instruction count each 16 slot separately?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89446