[fixup] Address review comments

Author: AGindinson

compiler/src/iree/compiler/Dialect/Flow/Transforms/RegionOpUtils.cpp

@@ -809,20 +809,21 @@ static bool isAttentionMaskGenerator(Operation *op) {
   return false;
 }
 
-static bool hasFusableOrLowMemoryUsers(Operation *op) {
-  bool hasMemoryHungryNonFusableConsumer = false;
+static bool hasExplicitNonFusableUsers(Operation *op) {
+  bool hasNonFusableUse = false;
   for (Operation *user : op->getUsers()) {
     if (isa<IREE::LinalgExt::LinalgFusionOpInterface>(user))
       continue;
-    // TODO: Calling iree_linalg_ext.scan ops "memory-hungry" is a bit of a
-    // stretch - the actual issue comes down to poor support in tiling configs,
-    // leading to overly large stack-bound allocations. Backend compilers should
-    // often cope with fusing the resulting loops.
-    // So improve the tiling logic for ScanOp's, while also considering ways to
-    // express simple fusions within this cumulative reduction intrinsic.
-    hasMemoryHungryNonFusableConsumer |= isa<IREE::LinalgExt::ScanOp>(user);
-  }
-  return !hasMemoryHungryNonFusableConsumer;
+    // TODO: The issue with iree_linalg_ext.scan compared to other non-fusable
+    // ops comes down to poor support in tiling configs, leading to overly large
+    // stack-bound allocations. In practice, backend compilers should often cope
+    // with fusing the resulting loops even without Linalg-level fusion.
+    // So long-term, we improve the tiling logic for ScanOp's, while also
+    // considering ways to express simple fusions within this cumulative
+    // reduction intrinsic.
+    hasNonFusableUse |= isa<IREE::LinalgExt::ScanOp>(user);
+  }
+  return hasNonFusableUse;
 }
 
 /// Operations that are cloned into dispatch regions formed with other
@@ -842,8 +843,8 @@ bool isClonableIntoDispatchOp(Operation *op,
     return true;
   }
   // TODO: Tune the cases excluded through hasFusableOrLowMemoryUsers
-  // condition in a more targetted manner, then remove the condition.
-  if (isa<linalg::LinalgOp>(op) && !hasFusableOrLowMemoryUsers(op)) {
+  // condition in a more targeted manner, then remove the condition.
+  if (isa<linalg::LinalgOp>(op) && hasExplicitNonFusableUsers(op)) {
     return false;
   }
   if (LinalgExt::isBitExtendOp(op)) {