[PATCH] D127853: [AArch64][SME] Add SME cntsb/h/w/d intrinsics

Wed Jun 15 10:57:02 PDT 2022

aemerson accepted this revision.
aemerson added a comment.
This revision is now accepted and ready to land.

In D127853#3585792 <https://reviews.llvm.org/D127853#3585792>, @david-arm wrote:

> In D127853#3585748 <https://reviews.llvm.org/D127853#3585748>, @aemerson wrote:
>
>> Can't the `rdsvl` immediate be used to do the scaling instead of a shift?
>
> I think the rdsvl immediate specifies the multiplier, which equates to a shift left. Whereas I want to divide the length by the element size, i.e. a shift right.

Ah yes. LGTM then.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127853