[PATCH] D158449: [IR]Add NumSrcElts param to is..Mask static function in ShuffleVectorInst.

Wed Oct 4 12:02:23 PDT 2023

ABataev added a comment.

In D158449#4653004 <https://reviews.llvm.org/D158449#4653004>, @reames wrote:

> Coming into this quite late, but I'm confused by the need for this API change.
>
> Doesn't the existing ArrayRef::take_front API fulfill the same need?  If you need to consider a prefix of the mask, why not just take a reference to that prefix?

Sure, it may work this way, but better to have this in the interface functions. It helped to find so many bugs in the cost model. Plus, this extract thing must be copied over almost all target-specific TTI implementations.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158449