[PATCH] D82227: SLP: honor requested max vector size merging PHIs

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 20 21:12:21 PDT 2020


rampitec added a comment.

In D82227#2105469 <https://reviews.llvm.org/D82227#2105469>, @ABataev wrote:

> Why it is required?


The whole pass only creates vectors up to the size of a vector register provided by a target, this is simply an omission in this place not to check it.

On practice the testcase is reduced from a real app where it has created two <32 x float> vectors and RA run out of registers. Both have to be alive at the same loop and we did not have sufficient number of registers.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82227/new/

https://reviews.llvm.org/D82227





More information about the llvm-commits mailing list