diff options
author | Alon Zakai <alonzakai@gmail.com> | 2016-11-04 17:24:01 -0700 |
---|---|---|
committer | Alon Zakai <alonzakai@gmail.com> | 2016-11-04 17:24:01 -0700 |
commit | 13ce3488333f70607411eb504ea2046fe2dc7c6d (patch) | |
tree | 6f7e8a6a529016dd7bb79d1c762d24e47d07bef2 /src | |
parent | fd5d26f549ec95f46d8696dac51ba5b5f7316248 (diff) | |
download | binaryen-13ce3488333f70607411eb504ea2046fe2dc7c6d.tar.gz binaryen-13ce3488333f70607411eb504ea2046fe2dc7c6d.tar.bz2 binaryen-13ce3488333f70607411eb504ea2046fe2dc7c6d.zip |
optimize booleans through i32.or
Diffstat (limited to 'src')
-rw-r--r-- | src/passes/OptimizeInstructions.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/passes/OptimizeInstructions.cpp b/src/passes/OptimizeInstructions.cpp index 6ba34990f..1970cf08f 100644 --- a/src/passes/OptimizeInstructions.cpp +++ b/src/passes/OptimizeInstructions.cpp @@ -363,8 +363,12 @@ private: } } } else if (auto* binary = boolean->dynCast<Binary>()) { - // x != 0 is just x if it's used as a bool - if (binary->op == NeInt32) { + if (binary->op == OrInt32) { + // an or flowing into a boolean context can consider each input as boolean + binary->left = optimizeBoolean(binary->left); + binary->right = optimizeBoolean(binary->right); + } else if (binary->op == NeInt32) { + // x != 0 is just x if it's used as a bool if (auto* num = binary->right->dynCast<Const>()) { if (num->value.geti32() == 0) { return binary->left; |