[PATCH] D44735: [LV] Add TTI::shouldMaximizeVectorBandwidth to allow enabling it per target

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 21 10:05:02 PDT 2018


kparzysz added a comment.

In https://reviews.llvm.org/D44735#1044638, @hsaito wrote:

> I suggest positioning https://reviews.llvm.org/D44735 as a simple refinement of https://reviews.llvm.org/D33341 and then add the above mentioned MinVF adjustment on top, as a separate patch.
>  Does that work?


This plan makes sense to me, but I'm not sure what you mean by the above.  Do you want me to reopen https://reviews.llvm.org/D33341?  The only change there was to enable the flag by default and update failing testcases, I'd like to keep it disabled by default.


Repository:
  rL LLVM

https://reviews.llvm.org/D44735





More information about the llvm-commits mailing list