[PATCH] D44574: [LV] Introduce TTI::getMinimumVF

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 21 07:16:50 PDT 2018


kparzysz added a comment.

In https://reviews.llvm.org/D44574#1044055, @hsaito wrote:

> This is a larger change than I was thinking. Making MaximizeBandwidth TTI based not good enough?


It sounds like a good idea to allow targets to enable it.  I published https://reviews.llvm.org/D44735 to do it.

The advantage of the approach in this patch is that it gives a guarantee of the minimum VF.  I'm not sure if the TTI callback for maximizing bandwidth will ensure that, since it still considers all powers of 2 between 1 and the MaxVF.


Repository:
  rL LLVM

https://reviews.llvm.org/D44574





More information about the llvm-commits mailing list