On Wed, 2019-02-13 at 20:08 +0100, Iain Buclaw wrote: > On Wed, 13 Feb 2019 at 14:37, Jakub Jelinek <ja...@redhat.com> wrote: > > > > On Wed, Feb 13, 2019 at 10:22:14AM +0000, Andrew Stubbs wrote: > > > On 13/02/2019 09:09, Jakub Jelinek wrote: > > > > To make it work together with doing llvm_binutils only once, > > > > the global now > > > > has multiple values > > > > 0 - disallow blank lines > > > > 1 - allow them for a single test only, reset after testing it > > > > in gcc-dg-prune > > > > 2 - allow it for all tests (llvm_binutils) > > > > > > FWIW, this scheme and patch looks good to me, although probably > > > the meaning > > > of the numbers ought to be documented somewhere in the code. > > > > Here is an updated patch that documents it. Bootstrapped/regtested > > on > > x86_64-linux and i686-linux, ok for trunk? > > > > I have no objections. > > Iain.
Likewise, LGTM (FWIW). Dave
