[PATCH] D97299: [IR][SVE] Add new llvm.experimental.stepvector intrinsic

Mon Mar 22 06:57:59 PDT 2021

fhahn added a comment.

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

> In D97299#2641273 <https://reviews.llvm.org/D97299#2641273>, @fhahn wrote:
>
>> In D97299#2641106 <https://reviews.llvm.org/D97299#2641106>, @david-arm wrote:
>>
>>> - Updated LangRef to better describe the intrinsic behaviour.
>>
>> Thanks. I think it should be possible to enforce the new restrictions in the IR verifier, to prevent users from accidentally using them with wrong types.
>
> Yeah that's right. The type (including the element size) are defended in this patch in Verifier.cpp.

That's great! It still looks like there are negative tests messing for the checks?

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

https://reviews.llvm.org/D97299