summaryrefslogtreecommitdiff
path: root/src/passes/OptimizeInstructions.cpp
diff options
context:
space:
mode:
authorAlon Zakai <azakai@google.com>2022-06-24 12:09:41 -0700
committerGitHub <noreply@github.com>2022-06-24 12:09:41 -0700
commitefea05006b4179db159d8850c33a4b54cf04d317 (patch)
treef0e0161924a1fcc7887e49fd5a88215932a1b3a6 /src/passes/OptimizeInstructions.cpp
parent10523fa2468a21b0add13ccfda2d42888be72040 (diff)
downloadbinaryen-efea05006b4179db159d8850c33a4b54cf04d317.tar.gz
binaryen-efea05006b4179db159d8850c33a4b54cf04d317.tar.bz2
binaryen-efea05006b4179db159d8850c33a4b54cf04d317.zip
[Wasm GC] OptimizeInstructions: Optimize ref.eq on equal inputs with a tee (#4749)
(ref.eq (local.tee $x (..)) (local.get $x) ) That will definitely return 1. Before this PR the side effects of tee stopped us from optimizing.
Diffstat (limited to 'src/passes/OptimizeInstructions.cpp')
-rw-r--r--src/passes/OptimizeInstructions.cpp13
1 files changed, 10 insertions, 3 deletions
diff --git a/src/passes/OptimizeInstructions.cpp b/src/passes/OptimizeInstructions.cpp
index f2ca4e0be..400ed6af0 100644
--- a/src/passes/OptimizeInstructions.cpp
+++ b/src/passes/OptimizeInstructions.cpp
@@ -25,6 +25,7 @@
#include <ir/abstract.h>
#include <ir/bits.h>
#include <ir/cost.h>
+#include <ir/drop.h>
#include <ir/effects.h>
#include <ir/eh-utils.h>
#include <ir/find_all.h>
@@ -1377,9 +1378,15 @@ struct OptimizeInstructions
skipCast(curr->right, Type::eqref);
// Identical references compare equal.
- if (areConsecutiveInputsEqualAndRemovable(curr->left, curr->right)) {
- replaceCurrent(
- Builder(*getModule()).makeConst(Literal::makeOne(Type::i32)));
+ // (Technically we do not need to check if the inputs are also foldable into
+ // a single one, but we do not have utility code to handle non-foldable
+ // cases yet; the foldable case we do handle is the common one of the first
+ // child being a tee and the second a get of that tee. TODO)
+ if (areConsecutiveInputsEqualAndFoldable(curr->left, curr->right)) {
+ auto* result =
+ Builder(*getModule()).makeConst(Literal::makeOne(Type::i32));
+ replaceCurrent(getDroppedChildrenAndAppend(
+ curr, *getModule(), getPassOptions(), result));
return;
}