Skip to content

Commit d39b88d

Browse files
committed
compiler: Revert incorrect modifications to DAG construction
1 parent 75847f8 commit d39b88d

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

devito/ir/support/basic.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,6 @@ def distance(self, other):
363363
# Case 1: `sit` is an IterationInterval with statically known
364364
# trip count. E.g. it ranges from 0 to 3; `other` performs a
365365
# constant access at 4
366-
# TODO: This case represents the majority of time constructing a DAG
367366
for v in (self[n], other[n]):
368367
try:
369368
if bool(v < sit.symbolic_min or v > sit.symbolic_max):
@@ -1123,10 +1122,6 @@ def d_anti(self):
11231122
"""Anti (or "write-after-read") dependences."""
11241123
return DependenceGroup(self.d_anti_gen())
11251124

1126-
@cached_property
1127-
def has_antidependencies(self):
1128-
return any(i.cause for i in self.d_anti_gen())
1129-
11301125
@memoized_generator
11311126
def d_output_gen(self):
11321127
"""Generate the output (or "write-after-write") dependences."""

devito/passes/clusters/misc.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -362,8 +362,9 @@ def is_cross(source, sink):
362362
# (intuitively, "the loop nests are to be kept separated")
363363
# * All ClusterGroups between `cg0` and `cg1` must precede `cg1`
364364
# * All ClusterGroups after `cg1` cannot precede `cg1`
365-
# TODO: Check that this is indeed what the attribute does
366-
if prefix and scope.has_antidependencies:
365+
366+
# FIXME: Slow
367+
if any(i.cause & prefix for i in scope.d_anti_gen()):
367368
for cg2 in cgroups[n:cgroups.index(cg1)]:
368369
dag.add_edge(cg2, cg1)
369370
for cg2 in cgroups[cgroups.index(cg1)+1:]:

0 commit comments

Comments
 (0)