[PATCH] D151782: Improve WebAssembly vector bitmask, mask reduction, and extending

Caleb Zulawski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 12:17:16 PDT 2023


calebzulawski added a comment.

In D151782#4406514 <https://reviews.llvm.org/D151782#4406514>, @tlively wrote:

> Sorry for the breakage! Taking a look now. If a fix looks nontrivial, I'll revert.

I've already identified the issue, the assert is triggering due to vectors of length 1. In that case, the combine step should be skipped.

Also, I noticed only the modified tests made it into the commit, not the new tests I added. Would you like me to open a new review with the changes? Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151782



More information about the llvm-commits mailing list