[PATCH] D99596: [RFC] [LoopDist] Distribute vectorizable loops

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 1 06:22:05 PDT 2021


SjoerdMeijer added a comment.

By just looking at this patch I find it a bit difficult to get an overview of all moving parts involved. I.e., this makes probably sense:

> Loop distribute bails out early if a loop is already vectorizable.

but by not doing this, do we remove opportunities for the vectoriser? So, perhaps the easiest is to get some perf numbers on the table?

Then, we can think about the cost-model too, and see if we can create some ideas about that. This pass is not enabled by default, so if perf numbers are okay and we don't make (downstream) users of this pass unhappy, it looks like a good step forward to me, but some ideas about steps after that would be good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99596



More information about the llvm-commits mailing list