[PATCH] D92059: [SLP] Control maximum vectorization factor from TTI

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 15:52:38 PST 2020


jonpa added a comment.

This LGTM, but for one little detail: "Maximum SLP vectorization factor" should perhaps include "0=unlimited" or something similar, to avoid confusing 0 to mean default "off". Or maybe that isn't needed with a hidden option?

@spatel looks good to you as well?


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

https://reviews.llvm.org/D92059



More information about the llvm-commits mailing list