On 5/21/20 11:01 PM, Jason Merrill wrote:
Why? What is the use of requiring ChangeLog entries at all for these changes?
I must confirm a common test-suite ChangeLog entry is something like:
$ grep ':' gcc/testsuite/ChangeLog | sed 's/.*://' | sort | uniq -c | sort -n |
tac | head -n 15
6309 Likewise.
1306 New test.
231 New.
68 New testcase.
55
54 Ditto.
47 New file.
33 Same.
27 Move into ...
24 New tests.
21 Add typedef for int32_t.
19 This. Clean up
17 Rename to...
16 This.
12 ...this.
$ grep ':' gcc/testsuite/ChangeLog-2019 | sed 's/.*://' | sort | uniq -c | sort
-n | tac | head -n 15
2981 Likewise.
2287 New test.
399 New testcase.
263 Same.
263 New.
217 Ditto.
111 Adjust.
88 New file.
62
32 Require exceptions.
30 Remove.
25 New
20 Update test.
17 New tests.
15 Add
$ grep ':' gcc/testsuite/ChangeLog-2018 | sed 's/.*://' | sort | uniq -c | sort
-n | tac | head -n 15
5071 Likewise.
2071 New test.
443 New.
413 New testcase.
395 Remove.
256 Ditto.
224 Same.
108 New file.
66 Likwise.
65 Adjust.
53
52 Dito.
39 Add -flinker-output=nolto-rel.
32 New test case.
26 Delete testcase.
So I'm open for relaxation of the rule.
What about the others?
Martin
