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

Thu Oct 15 13:54:45 PDT 2020

tlively marked an inline comment as done.
tlively added inline comments.

================
Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:117

+TARGET_BUILTIN(__builtin_wasm_popcnt, "V16ScV16Sc", "nc", "simd128")
+
----------------
aheejin wrote:
> - 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?
Yes, the it does a popcnt on each 1-byte lane individually.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89446