[PATCH] D44523: Change calculation of MaxVectorSize

[Hideki Saito via Phabricator via llvm-commits]

hsaito added a comment.

In https://reviews.llvm.org/D44523#1041288, @Ayal wrote:

> See **`MaximizeBandwidth`**, as in
>  llvm-dev's Enable vectorizer-maximize-bandwidth by default? <http://lists.llvm.org/pipermail/llvm-dev/2017-June/113581.html>
>  patch: Enable vectorizer-maximize-bandwidth by default. <https://reviews.llvm.org/D33341>
>  which is still reverted afaik: r306936 - Revert "r306473 - re-commit r306336: Enable vectorizer-maximize-bandwidth by default." <http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20171211/509910.html>


Thanks, Ayal. I somehow thought that attempt was TTI based, not just the flip of the default for all targets at once. I think we should enable it per target basis.


Repository:
  rL LLVM

https://reviews.llvm.org/D44523