From 87d8d0e1f25c7dc4f6d75d0af2d92c5163f6a05c Mon Sep 17 00:00:00 2001 From: Mathias Louboutin Date: Mon, 18 Sep 2023 08:25:17 -0400 Subject: [PATCH] compiler: remove atomic collapse hack --- devito/passes/iet/parpragma.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/devito/passes/iet/parpragma.py b/devito/passes/iet/parpragma.py index 4af585f86a..9d69e12df7 100644 --- a/devito/passes/iet/parpragma.py +++ b/devito/passes/iet/parpragma.py @@ -295,9 +295,6 @@ def _select_candidates(self, candidates): except TypeError: pass - # At least one inner loop (nested) or - # we do not collapse most inner loop if it is an atomic reduction - if not i.is_ParallelAtomic or nested: collapsable.append(i) # Give a score to this candidate, based on the number of fully-parallel @@ -429,11 +426,6 @@ def _make_nested_partree(self, partree): if self.nhyperthreads <= self.nested: return partree - # Loop nest with atomic reductions are more likely to have less latency - # keep outer loop parallel - if partree.root.is_ParallelAtomic: - return partree - # Note: there might be multiple sub-trees amenable to nested parallelism, # hence we loop over all of them #