SjoerdMeijer added a comment. Hi Florian, thanks for your input!
> IMO it would make sense to have the more specific pragmas imply > vectorize(enable) here (or update the docs accordingly). Yep, fully agree with that, as I also wrote in my previous comment. And thanks for digging up that PR. I think that supports this case and that the proposed behaviour in this patch is what one would expect. If we agree on that, I wouldn't mind fixing that PR too as that seems low hanging-fruit to me. And to be honest, at the moment I have no clue what the meaning is, or could/should be, when a more specific pragma like vector_width or vector_predicate is set, without also setting/implying vectorize(enable). I need to look into that... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65776/new/ https://reviews.llvm.org/D65776 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
