[PATCH] D111919: [RISCV][WebAssembly][TargetLowering] Allow expandCTLZ/expandCTTZ to rely on CTPOP expansion for vectors.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 18 18:37:43 PDT 2021


craig.topper added a comment.

In D111919#3071452 <https://reviews.llvm.org/D111919#3071452>, @tlively wrote:

> In D111919#3071315 <https://reviews.llvm.org/D111919#3071315>, @craig.topper wrote:
>
>> In D111919#3071254 <https://reviews.llvm.org/D111919#3071254>, @tlively wrote:
>>
>>> Hmm yeah the new lowerings for WebAssembly don't look great. Would there be a way to disable this change for Wasm?
>>
>> Would you want ctpop to go to scalarization as well?
>
> Yeah actually that would probably be fine if it makes things simpler.

I'm not sure if it is easier or not. I just thought it might be better for overall consistency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111919



More information about the llvm-commits mailing list