SjoerdMeijer added a comment. I am very sorry that I am late to this... but I do have some concerns.
The concern that I have is that we extend vecorize_width with a scalable/fixed boolean, but there are more vectorisation pragma that set vectorisation options which imply enabling vectorisation: > Pragmas setting transformation options imply the transformation is enabled, > as if it was enabled via the corresponding transformation pragma (e.g. > vectorize(enable)) Thus, unless I miss something, I don't think this should be an option to vectorize_width, but to me it looks like we need a separate one, e.g.: vectorize_scalable(enable|disable) what do you think? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89031/new/ https://reviews.llvm.org/D89031 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
