[PATCH] D44523: Change calculation of MaxVectorSize

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 13:00:48 PDT 2018


hsaito added a comment.

In https://reviews.llvm.org/D44523#1039300, @kparzysz wrote:

> What would work well for us too would to enable pumping for targets that want it, but leave it off by default.  This wouldn't change any existing behavior, but would help HVX a great deal.  In any case I'd be interested to hear your feedback on this, if you have any other suggestions on how to deal with the short vector issue.


I think Michael Kuperstein was trying to enable double pumping (could be one year or more ago). I think it's best to dig it up. There may be something usable in there.


Repository:
  rL LLVM

https://reviews.llvm.org/D44523





More information about the llvm-commits mailing list