[PATCH] D15679: [FIX] Schedule generation PR25879

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 6 05:05:47 PST 2016


jdoerfert added a comment.

In http://reviews.llvm.org/D15679#320281, @grosser wrote:

> Hi Johannes,
>
> I played around with this code a little bit more and it seems we can pull out the tree traversal to remove the single-non-affine-region special case and generally simplify the code. This helps me to better understand that we _either_ walk the tree in case we have an affine-region-node or we build schedule information for a ScopStmt.
>
> F1284398: 0001-Define-buildScheduleRec-on-RegionNodes-and-pull-out-.patch <http://reviews.llvm.org/F1284398>


I commited a refactoring in r256931. It is different to your's but I think it makes your point even more clear. I hope you agree.


http://reviews.llvm.org/D15679





More information about the llvm-commits mailing list