Allow writing mutators with lambdas

src/Bounds.cpp

@@ -2744,23 +2744,13 @@ class BoxesTouched : public IRGraphVisitor {
     }
 
     vector<const Variable *> find_free_vars(const Expr &e) {
-        class FindFreeVars : public IRVisitor {
-            using IRVisitor::visit;
-            void visit(const Variable *op) override {
-                if (scope.contains(op->name)) {
-                    result.push_back(op);
-                }
+        vector<const Variable *> result;
+        visit_with(e, [&](const Variable *op, auto *) {
+            if (scope.contains(op->name)) {
+                result.push_back(op);
             }
-
-        public:
-            const Scope<Interval> &scope;
-            vector<const Variable *> result;
-            FindFreeVars(const Scope<Interval> &s)
-                : scope(s) {
-            }
-        } finder(scope);
-        e.accept(&finder);
-        return finder.result;
+        });
+        return result;
     }
 
     void visit(const IfThenElse *op) override {

src/BoundsInference.cpp

@@ -34,40 +34,30 @@ bool var_name_match(const string &candidate, const string &var) {
     return (candidate == var) || Internal::ends_with(candidate, "." + var);
 }
 
-class DependsOnBoundsInference : public IRVisitor {
-    using IRVisitor::visit;
-
-    void visit(const Variable *var) override {
-        if (ends_with(var->name, ".max") ||
-            ends_with(var->name, ".min")) {
-            result = true;
-        }
-    }
-
-    void visit(const Call *op) override {
-        if (op->name == Call::buffer_get_min ||
-            op->name == Call::buffer_get_max) {
-            result = true;
-        } else {
-            IRVisitor::visit(op);
-        }
-    }
-
-public:
-    bool result = false;
-    DependsOnBoundsInference() = default;
-};
-
 bool depends_on_bounds_inference(const Expr &e) {
-    DependsOnBoundsInference d;
-    e.accept(&d);
-    return d.result;
+    bool result = false;
+    visit_with(
+        e,
+        [&](const Variable *var, auto *) {
+            if (ends_with(var->name, ".max") ||
+                ends_with(var->name, ".min")) {
+                result = true;
+            }  //
+        },
+        [&](const Call *op, auto *self) {
+            if (op->name == Call::buffer_get_min ||
+                op->name == Call::buffer_get_max) {
+                result = true;
+            } else {
+                self->visit_base(op);
+            }  //
+        });
+    return result;
 }
 
 /** Compute the bounds of the value of some variable defined by an
  * inner let stmt or for loop. E.g. for the stmt:
  *
- *
  * for x from 0 to 10:
  *  let y = x + 2;
  *

src/Function.cpp

@@ -1250,50 +1250,36 @@ const Call *Function::is_wrapper() const {
     }
 }
 
-namespace {
-
-// Replace all calls to functions listed in 'substitutions' with their wrappers.
-class SubstituteCalls : public IRMutator {
-    using IRMutator::visit;
-
-    const map<FunctionPtr, FunctionPtr> &substitutions;
-
-    Expr visit(const Call *c) override {
-        Expr expr = IRMutator::visit(c);
-        c = expr.as<Call>();
-        internal_assert(c);
-
-        if ((c->call_type == Call::Halide) &&
-            c->func.defined() &&
-            substitutions.count(c->func)) {
-            auto it = substitutions.find(c->func);
-            internal_assert(it != substitutions.end())
-                << "Function not in environment: " << c->func->name << "\n";
-            FunctionPtr subs = it->second;
-            debug(4) << "...Replace call to Func \"" << c->name << "\" with "
-                     << "\"" << subs->name << "\"\n";
-            expr = Call::make(c->type, subs->name, c->args, c->call_type,
-                              subs, c->value_index,
-                              c->image, c->param);
-        }
-        return expr;
-    }
-
-public:
-    SubstituteCalls(const map<FunctionPtr, FunctionPtr> &substitutions)
-        : substitutions(substitutions) {
-    }
-};
-
-}  // anonymous namespace
-
 Function &Function::substitute_calls(const map<FunctionPtr, FunctionPtr> &substitutions) {
     debug(4) << "Substituting calls in " << name() << "\n";
     if (substitutions.empty()) {
         return *this;
     }
-    SubstituteCalls subs_calls(substitutions);
-    contents->mutate(&subs_calls);
+
+    // Replace all calls to functions listed in 'substitutions' with their wrappers.
+    auto m = LambdaMutator{
+        [&](const Call *c, auto *self) {
+            Expr expr = self->visit_base(c);
+            c = expr.as<Call>();
+            internal_assert(c);
+
+            if ((c->call_type == Call::Halide) &&
+                c->func.defined() &&
+                substitutions.count(c->func)) {
+                auto it = substitutions.find(c->func);
+                internal_assert(it != substitutions.end())
+                    << "Function not in environment: " << c->func->name << "\n";
+                FunctionPtr subs = it->second;
+                debug(4) << "...Replace call to Func \"" << c->name << "\" with "
+                         << "\"" << subs->name << "\"\n";
+                expr = Call::make(c->type, subs->name, c->args, c->call_type,
+                                  subs, c->value_index,
+                                  c->image, c->param);
+            }
+            return expr;
+        }};
+
+    contents->mutate(&m);
     return *this;
 }
 

src/IR.h

@@ -23,6 +23,16 @@ namespace Internal {
 
 class Function;
 
+// The lambda-based IRVisitors and IRMutators use this helper to dispatch
+// to multiple lambdas via function overloading (of `operator()`).
+template<typename... Ts>
+struct LambdaOverloads : Ts... {
+    using Ts::operator()...;
+    explicit LambdaOverloads(Ts... ts)
+        : Ts(std::move(ts))... {
+    }
+};
+
 /** The actual IR nodes begin here. Remember that all the Expr
  * nodes also have a public "type" property */