[PATCH] D144128: [SLP] Check with target before vectorizing GEP Indices

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a comment.

It would be ok for me if you reverted it while investigating regressions. I would hope then that you would try the "partial revert" meaning removing just the change in NotProfitableForVectorization().

Since targets are different and cost tuning is coarse and difficult, maybe it would make sense to change this hook so that there are different levels of preference. It should be possible to use it as is, but for your targets you could keep using it but only where you wanted to...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144128