[PATCH] D102748: [LoopUnroll] Don't unroll before vectorisation

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Thanks for those examples. I am going to play a little bit more with this.

In terms of a way forward (and getting the example properly vectorised):

- I am going to try and move the vectoriser passes to just before the full unroller. I am somewhat sceptical, i.e. expecting a lot of fall out, but let's see.
- I don't think I see any potential in tuning the full unroller and let it unroll less because it sounds fragile.
- and I don't think there's an easy way forward with this in the SLP vectoriser.

So what remains is playing with moving the vectoriser earlier. But more/other ideas are more than welcome.


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

https://reviews.llvm.org/D102748