From c06b2925232b92fd379a3e8d513ab361b8434a15 Mon Sep 17 00:00:00 2001 From: Alon Zakai Date: Mon, 11 Oct 2021 17:37:34 -0700 Subject: 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. --- src/passes/LegalizeJSInterface.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/passes/LegalizeJSInterface.cpp') 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) { -- cgit v1.2.3