Refactor interaction between Pass and PassRunner (#5093)

Previously only WalkerPasses had access to the `getPassRunner` and
`getPassOptions` methods. Move those methods to `Pass` so all passes can use
them. As a result, the `PassRunner` passed to `Pass::run` and
`Pass::runOnFunction` is no longer necessary, so remove it.

Also update `Pass::create` to return a unique_ptr, which is more efficient than
having it return a raw pointer only to have the `PassRunner` wrap that raw
pointer in a `unique_ptr`.

Delete the unused template `PassRunner::getLast()`, which looks like it was
intended to enable retrieving previous analyses and has been in the code base
since 2015 but is not implemented anywhere.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/passes/DuplicateImportElimination.cpp b/src/passes/DuplicateImportElimination.cpp
index 1a0319e3a..33ce2288a 100644
--- a/src/passes/DuplicateImportElimination.cpp
+++ b/src/passes/DuplicateImportElimination.cpp
@@ -31,7 +31,7 @@ struct DuplicateImportElimination : public Pass {
   // This pass does not alter function contents.
   bool requiresNonNullableLocalFixups() override { return false; }
 
-  void run(PassRunner* runner, Module* module) override {
+  void run(Module* module) override {
     ImportInfo imports(*module);
     std::map<Name, Name> replacements;
     std::map<std::pair<Name, Name>, Name> seen;
@@ -54,7 +54,7 @@ struct DuplicateImportElimination : public Pass {
     }
     if (!replacements.empty()) {
       module->updateMaps();
-      OptUtils::replaceFunctions(runner, *module, replacements);
+      OptUtils::replaceFunctions(getPassRunner(), *module, replacements);
       for (auto name : toRemove) {
         module->removeFunction(name);
       }