Skip to content

Commit

Permalink
Fix up fuzzing test on Windows.
Browse files Browse the repository at this point in the history
3bd112c fixed the fuzzing test on Linux, which, after
https://reviews.llvm.org/D125933, has one less branch. Turns out, on
Windows, that it still has the extra branch. I'm guessing that's because
exit() isn't known to be noreturn on Windows or something.

Either way, just make the test more tolerant.
  • Loading branch information
hctim committed May 20, 2022
1 parent f4570ce commit 80ac0b9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions compiler-rt/test/fuzzer/merge_two_step.test
Expand Up @@ -10,7 +10,7 @@ RUN: echo ..Z... > %t/T1/3
RUN: rm -f %t/MCF
RUN: %run %t-FullCoverageSetTest -merge=1 -merge_control_file=%t/MCF %t/T0 %t/T1 2>&1 | FileCheck %s --check-prefix=CHECK1
CHECK1: MERGE-OUTER: 3 files, 0 in the initial corpus
CHECK1: MERGE-OUTER: 3 new files with {{.*}} new features added; 10 new coverage edges
CHECK1: MERGE-OUTER: 3 new files with {{.*}} new features added; {{.*}} new coverage edges

RUN: echo ...Z.. > %t/T2/1
RUN: echo ....E. > %t/T2/2
Expand All @@ -28,4 +28,4 @@ CHECK2: MERGE-OUTER: starting merge from scratch, but reusing coverage informati
CHECK2: MERGE-OUTER: 7 files, 0 in the initial corpus, 3 processed earlier
CHECK2: MERGE-INNER: using the control file
CHECK2: MERGE-INNER: 4 total files; 0 processed earlier; will process 4 files now
CHECK2: MERGE-OUTER: 6 new files with {{.*}} new features added; 13 new coverage edges
CHECK2: MERGE-OUTER: 6 new files with {{.*}} new features added; {{.*}} new coverage edges

0 comments on commit 80ac0b9

Please sign in to comment.