|
8 | 8 | // REQUIRES: linux |
9 | 9 | // |
10 | 10 | // Explanation of the command: |
11 | | -// - search for all "XFAIL" occurences, display line with match and the next one |
| 11 | +// - search for all "XFAIL" occurrences, display line with match and the next one |
12 | 12 | // -I, --include to drop binary files and other unrelated files |
13 | | -// - in the result, serach for "XFAIL" again, but invert the result - this |
| 13 | +// - in the result, search for "XFAIL" again, but invert the result - this |
14 | 14 | // allows us to get the line *after* XFAIL |
15 | 15 | // - in those lines, check that XFAIL-TRACKER is present and correct. Once |
16 | 16 | // again, invert the search to get all "bad" lines |
|
23 | 23 | // RUN: grep -Pv "XFAIL-TRACKER:\s+.*intel/llvm.*\d+" | \ |
24 | 24 | // RUN: wc -l | FileCheck %s --check-prefix NUMBER-OF-XFAIL-WITHOUT-TRACKER |
25 | 25 | // |
26 | | -// The number below is a number of tests which are *impropertly* XFAIL-ed, i.e. |
| 26 | +// The number below is a number of tests which are *improperly* XFAIL-ed, i.e. |
27 | 27 | // we either don't have a tracker associated with a failure listed in those |
28 | 28 | // tests, or it is listed in a wrong format. |
29 | 29 | // Note: strictly speaking, that is not amount of files, but amount of XFAIL |
30 | 30 | // directives. If a test contains several XFAIL directives, some of them may be |
31 | 31 | // valid and other may not. |
32 | 32 | // |
33 | | -// That number *must not* increase. Any PR which causes this number to group |
| 33 | +// That number *must not* increase. Any PR which causes this number to grow |
34 | 34 | // should be rejected and it should be updated to either keep the number as-is |
35 | 35 | // or have it reduced (preferrably, down to zero). |
36 | 36 | // |
37 | 37 | // If you see this test failed for your patch, it means that you either |
38 | 38 | // introduced XFAIL directive to a test improperly, or broke the format of an |
39 | 39 | // existing XFAIL-ed tests. |
40 | 40 | // Another possibility (and that is a good option) is that you updated some |
41 | | -// tests to match the requried format and in that case you should just update |
| 41 | +// tests to match the required format and in that case you should just update |
42 | 42 | // (i.e. reduce) the number below. |
43 | 43 | // |
44 | 44 | // NUMBER-OF-XFAIL-WITHOUT-TRACKER: 176 |
0 commit comments