[PATCH] D88962: [SVE] Add support for scalable vectors with vectorize.scalable.enable loop attribute

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

In D88962#2422190 <https://reviews.llvm.org/D88962#2422190>, @SjoerdMeijer wrote:

> What do we do when vectorize.scalable is not supported by the target? We want to issue a diagnostic/remark? Do we need to say something about this in the LangRef part?

Yes, although I think that's worth addressing separately, I made a similar comment on D91718 <https://reviews.llvm.org/D91718> where such functionality is needed.  If that's addressed separately, are you happy @david-arm to land this patch @SjoerdMeijer?

(it is worth noting that this patch still ignores the 'scalable' VF in the vectorizer. This is only enabled again in D91077 <https://reviews.llvm.org/D91077>, which has a dependence on this patch)


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

https://reviews.llvm.org/D88962