[PATCH] D41953: [LoopUnroll] Unroll and Jam
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 21 07:24:30 PDT 2018
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
More information about the llvm-commits
mailing list