[PATCH] D89031: [SVE] Add support to vectorize_width loop pragma for scalable vectors

[Sjoerd Meijer via Phabricator via cfe-commits]

SjoerdMeijer added a comment.

Because I was not understanding, we have discussed this further offline.

I think the conclusion was: pragma `vectorize_width` controls the vectorisation vector `VF` in `<vscale x VF x elty>`. where `vscale` is not just a separate thing but it defines a VectorType. That's why it would make sense to attach `scalable|fixed` to `vectorize_width`. I agree with this and seems reasonable.

I still don't see how the proposed extension here allows you to specify fixed width vectorisation for:

  #pragma clang loop interleave_count(4)

targeting SVE and would appreciate if someone can comment on this example, but won't be holding up this work.


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

https://reviews.llvm.org/D89031