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

Paul Walker via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 18 04:48:52 PST 2020


paulwalker-arm added a comment.

As I see it there are a bunch of pragmas that all enable vectorisation, with each pragma providing a unit of information.  One component of this information is the vectorisation factor hint provided by vectorize_width.

With the introduction of scalable vectors this hint is using the wrong datatype and thus needs to be updated to allow `vectorize_width(#num,[fixed|scalable])` and `vectorize_width([fixed|scalable])` along side the existing `vectorize_width(#num)` representation that effectively becomes an alias to `vectorize_width(#num, fixed)`.

Doing this means all existing usages work as expected and there's now extra power to better guide the chosen vectorisation factor.


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

https://reviews.llvm.org/D89031



More information about the cfe-commits mailing list