Add runOnModuleCode helper. NFC (#4234)

This method is in parallel to runOnFunction above it. It sets the runner
and then does the walk, like that method.

Also set runner to nullptr by default. I noticed ubsan was warning on
things here, which this should avoid, but otherwise I'm not aware of an
actual bug, so this should be NFC. But it does provide a safer API
that should avoid future bugs.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/passes/LegalizeJSInterface.cpp b/src/passes/LegalizeJSInterface.cpp
index e7a5f9220..e5a69beb1 100644
--- a/src/passes/LegalizeJSInterface.cpp
+++ b/src/passes/LegalizeJSInterface.cpp
@@ -143,7 +143,7 @@ struct LegalizeJSInterface : public Pass {
 
       Fixer fixer(&illegalImportsToLegal);
       fixer.run(runner, module);
-      fixer.walkModuleCode(module);
+      fixer.runOnModuleCode(runner, module);
 
       // Finally we can remove all the now-unused illegal imports
       for (const auto& pair : illegalImportsToLegal) {