This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, contrib] Support multi-tool sum files in dg-cmp-results.sh
On 19 June 2017 at 12:44, Thomas Preudhomme
> Wrong copy paste between the patch I tested and the patch I sent. The first
> and second command of the sed should be replaced, not the second and third
> as in the patch I sent. For more safety I'll rerun the tests.
OK great. If you send an updated patch, I can re-run my tests easily.
> Best regards,
> On 15/06/17 17:15, Thomas Preudhomme wrote:
>> Forgetting the patch: check!
>> Sending it later as a reply to the wrong message: check!
>> Hopefully I won't check a second time any of those.
>> Best regards,
>> On 14/06/17 13:30, Thomas Preudhomme wrote:
>>> dg-cmp-results.sh contrib script is written to work with sum file for
>>> a single tool only. It throws away the header including the first ===
>>> line and everything starting from the following ===, assuming it is the
>>> test result. This does not work well for sum files with results for
>>> multiple tools.
>>> This patch changes the logic to instead keep everything between "Running
>>> target" line and the beginning of Summary line. Other existing filter
>>> mechanism will ensure only FAIL, PASS, etc. lines are kept after that.
>>> ChangeLog entry is as follow:
>>> *** contrib/ChangeLog ***
>>> 2017-06-14 Thomas Preud'homme <email@example.com>
>>> * dg-cmp-results.sh: Keep test result lines rather than throwing
>>> header and summary to support sum files with multiple tools.
>>> Tested successfully on sum file with single tool with similar results
>>> and on sum file with multiple tools now showing a regression with patch
>>> proposed in https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00875.html
>>> Is this ok for trunk?
>>> Best regards,