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

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

Hi @david-arm, can you document the reasons for the change?

Personally, I don't care much about the details of the metadata layout so if this switched layout represents value then great.  However, I'm less happy with the proposed pragma change because there's been a lot of effort spent (with plenty more to come) converting existing fixed length only representations of vectors into new types that can represent both fixed and scalable vectors.  To then force this information to be split into its constitute parts at the user level (i.e. pragmas and common line options) seems like a backward step.


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

https://reviews.llvm.org/D88962