diff options
| author | Alon Zakai <alonzakai@gmail.com> | 2019-04-10 10:16:57 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-04-10 10:16:57 -0700 |
| commit | 7cc509f54a759034fbff57fae64e142ad15cc097 (patch) | |
| tree | 9311e086d02a645707dbb5692119a5450e52b6c6 /src/ir/ExpressionManipulator.cpp | |
| parent | b13db5a0bc1170494ba845ab66129a506b251fde (diff) | |
| download | binaryen-7cc509f54a759034fbff57fae64e142ad15cc097.tar.gz binaryen-7cc509f54a759034fbff57fae64e142ad15cc097.tar.bz2 binaryen-7cc509f54a759034fbff57fae64e142ad15cc097.zip | |
Fuzz fixes (#1991)
Get fuzzer to attempt to create almost all features. Pass v8 all the flags to allow that.
Fix fuzz bugs where we read signed_ even when it was irrelevant for that type of load.
Improve wasm-reduce on fuzz testcases, try to replace a node with drops of its children, not just the children themselves.
Diffstat (limited to 'src/ir/ExpressionManipulator.cpp')
| -rw-r--r-- | src/ir/ExpressionManipulator.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/ir/ExpressionManipulator.cpp b/src/ir/ExpressionManipulator.cpp index 578d35e3f..2b648e077 100644 --- a/src/ir/ExpressionManipulator.cpp +++ b/src/ir/ExpressionManipulator.cpp @@ -14,6 +14,7 @@ * limitations under the License. */ +#include "ir/load-utils.h" #include "ir/utils.h" #include "support/hash.h" @@ -91,7 +92,7 @@ Expression* flexibleCopy(Expression* original, Module& wasm, CustomCopier custom return builder.makeAtomicLoad(curr->bytes, curr->offset, copy(curr->ptr), curr->type); } - return builder.makeLoad(curr->bytes, curr->signed_, curr->offset, curr->align, copy(curr->ptr), curr->type); + return builder.makeLoad(curr->bytes, LoadUtils::isSignRelevant(curr) ? curr->signed_ : false, curr->offset, curr->align, copy(curr->ptr), curr->type); } Expression* visitStore(Store *curr) { if (curr->isAtomic) { |