Philipp Schaad
Philipp Schaad
Leaving a note to revisit when https://github.com/spcl/dace/pull/1676 is merged.
@acalotoiu ping for re-review.
Thank you for continuing this @pratyai :-) As with #1639, I would prefer to briefly discuss this in the DaCe meeting before accepting, for the same reasons listed in #1639....
Leaving a note here to revisit this transformation for slight adaptation after https://github.com/spcl/dace/pull/1676 is merged.
It seems that the way the work-depth analysis for an SDFG was implemented, it is a destructive analysis (breaking loop cycles). I will indirectly fix this by transitioning the analysis...
@tbennun I see, that makes sense. I was expecting this to be a non-issue in 1.x, but I suppose we will need to fix `newast.py` for 1.0 then.
@tbennun ping
@acalotoiu What's the status here, are we still waiting for something on this PR?
Pinging @tbennun for re-review about the mentioned minor change