summaryrefslogtreecommitdiff
path: root/src/tools/wasm-opt.cpp
diff options
context:
space:
mode:
authorAlon Zakai <azakai@google.com>2022-08-24 09:56:23 -0700
committerGitHub <noreply@github.com>2022-08-24 09:56:23 -0700
commit892ebbe8a11417d0ffd3a5c798fb8bdab89d6634 (patch)
tree0c4b30aee02211a9ded29358db765f45e5424c28 /src/tools/wasm-opt.cpp
parent594ff7b9609656edb83187cb4600b23b3f2fde37 (diff)
downloadbinaryen-892ebbe8a11417d0ffd3a5c798fb8bdab89d6634.tar.gz
binaryen-892ebbe8a11417d0ffd3a5c798fb8bdab89d6634.tar.bz2
binaryen-892ebbe8a11417d0ffd3a5c798fb8bdab89d6634.zip
Fuzzer simplification: Remove trap-ignoring logic (#4958)
The "ignore trap" logic there is not close to enough for what we'd need to actually fuzz in a way that ignores traps, so this removes it. Atm that logic just allows a trap to happen without causing an error (that is, when comparing two results, one might trap and the other not, but they'd still be considered "equal"). But due to how we optimize traps in TrapsNeverHappens mode, the optimizer is free to assume the trap never occurs, which might remove side effects that are noticeable later. To actually handle that, we'd need to refactor the code to retain results per function (including the Loggings) and then to ignore everything from the very first trapping function. That is somewhat complicated to do, and a simpler thing is done in #4936, so we won't need it here.
Diffstat (limited to 'src/tools/wasm-opt.cpp')
-rw-r--r--src/tools/wasm-opt.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tools/wasm-opt.cpp b/src/tools/wasm-opt.cpp
index 3927e513c..538496ecf 100644
--- a/src/tools/wasm-opt.cpp
+++ b/src/tools/wasm-opt.cpp
@@ -314,7 +314,7 @@ int main(int argc, const char* argv[]) {
runner.run();
}
- ExecutionResults results(options.passOptions);
+ ExecutionResults results;
if (fuzzExecBefore) {
results.get(wasm);
}