[PATCH] D44523: Change calculation of MaxVectorSize

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 16 10:43:11 PDT 2018


hsaito added a comment.

In https://reviews.llvm.org/D44523#1039543, @dcaballe wrote:

> Maybe it's just a question of changing `MaxVectorSize` as suggested, and refine the cost modeling, if necessary.


The key here is moving to that one target at a time, i.e., when that target is ready --- and TTI is a good way of doing so.
So, if someone wants to change this "MaxVectorSize" determination TTI based, I'd support the idea.


Repository:
  rL LLVM

https://reviews.llvm.org/D44523





More information about the llvm-commits mailing list