On 01/25/2016 09:13 PM, David Malcolm wrote:
Here's an updated version of the patch.

Thanks!

Instead of testing one particular kind of output via a plugin,
this version of the patch adds code to gcc-dg-prune to issue a
FAIL for any testcase containing blank lines, with a new
   dg-allow-blank-lines-in-output
directive for those test cases that legimitately emit blank lines.
Examples of the latter include a test using -ftime-report, another
using -fdump-tree-cunrolli-details=stderr, and a Fortran test
using -fdump-fortran-original.

Is the =stderr test really necessary, or does it somehow predate the ability to scan dumps?

OK for trunk in stage 4?  I regard PR 69006 as a regression, and it
affects all diagnostics we output (unless caret-printing is
disabled).

Yes, I think so. Ok.

+  for (int row = layout.get_first_line ();
+       row <= last_line;
+       row++)

While you're fixing the layout here, see if that doesn't all fit on one line.


Bernd

Reply via email to