[PATCH] D41953: [LoopUnroll] Unroll and Jam

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

This looked very reasonable to me as an initial commit. And I thought so too, like Hal and Eric, that different enhancements can still be made. But as I wrote, we can iterate further on this once we've got something in. So I don't think this was a misstep, and once the buildbot failures are resolved, it looks like this can be recommitted. However, as I also mentioned, another reason for an initial commit is more exposure, potentially more people looking at it. So this discussion is great, the more feedback, the better; especially if there's something that needs to be addressed before a recommit.


Repository:
  rL LLVM

https://reviews.llvm.org/D41953