[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