[PATCH] D41953: [LoopUnroll] Unroll and Jam

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Hi Dave,

I haven't looked at all the nitty gritty details yet, but had some general questions after a first glance:

- this looks like a rather complete implementation (as opposed to some WIP prototype). In your description you mentioned something about testing. I was wondering if you can elaborate a little bit on this, if you had some more experiences with this pass since posting it. E.g., you've obviously added regression tests, but does it trigger also on the test-suite, etc.?
- probably the other thing people first want to know if this is (always) beneficial for their back-end. Do you have some data/experience with this?


Repository:
  rL LLVM

https://reviews.llvm.org/D41953