diff options
| author | Alon Zakai <alonzakai@gmail.com> | 2018-11-15 10:06:39 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-11-15 10:06:39 -0800 |
| commit | 622cba9c49378f89ca94e05cbbe2af0b0c62ab44 (patch) | |
| tree | 25d4c0d3837fb8b0f84cd2eebee1414011d7c289 /src/passes/pass.cpp | |
| parent | 892d2d747883aca132d2eb4e3ba5a6b261f944af (diff) | |
| download | binaryen-622cba9c49378f89ca94e05cbbe2af0b0c62ab44.tar.gz binaryen-622cba9c49378f89ca94e05cbbe2af0b0c62ab44.tar.bz2 binaryen-622cba9c49378f89ca94e05cbbe2af0b0c62ab44.zip | |
Don't try to optimize away unused names in RemoveUnusedBrs (#1750)
Rely on the dedicated pass for that. It's not worth the extra complexity to try, as we can't easily handle all the cases anyhow.
Add another run of the dedicated name-removing pass in the default passes.
Diffstat (limited to 'src/passes/pass.cpp')
| -rw-r--r-- | src/passes/pass.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/passes/pass.cpp b/src/passes/pass.cpp index 577ccd36a..26f771277 100644 --- a/src/passes/pass.cpp +++ b/src/passes/pass.cpp @@ -184,7 +184,8 @@ void PassRunner::addDefaultFunctionOptimizationPasses() { add("code-folding"); } add("merge-blocks"); // makes remove-unused-brs more effective - add("remove-unused-brs"); // coalesce-locals opens opportunities for optimizations + add("remove-unused-brs"); // coalesce-locals opens opportunities + add("remove-unused-names"); // remove-unused-brs opens opportunities add("merge-blocks"); // clean up remove-unused-brs new blocks add("optimize-instructions"); // late propagation |