[PATCH] D106506: [WebAssembly] Replace @llvm.wasm.popcnt with @llvm.ctpop.v16i8

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 21 16:41:34 PDT 2021


tlively added inline comments.


================
Comment at: llvm/test/CodeGen/WebAssembly/simd-unsupported.ll:13
 ; CHECK-LABEL: ctlz_v16i8:
-; CHECK: i32.clz
+; CHECK: i8x16.popcnt
 declare <16 x i8> @llvm.ctlz.v16i8(<16 x i8>, i1)
----------------
aheejin wrote:
> Aren't these now... "supported"? (This file's name is simd-unsupported.ll, so...)
No, I would say these are still unsupported because they don't map down to a single SIMD instruction. The i8x16.popcnt is still just part of a rather large expansion of the operation. Having i8x16.popcnt available makes the expansion better than before, though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106506



More information about the llvm-commits mailing list