Re: PR target/78213 revisited (was Re: [PATCH 5/9] Introduce selftest::locate_file (v4))

2016-12-14 Thread Bernd Schmidt
On 12/09/2016 08:32 PM, David Malcolm wrote: Thanks. Unfortunately, applying the "locate_file" patch https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01186.html would now introduce a regression in a recently-added test case: The problem is that this DejaGnu test case uses -fself-test, and does

PR target/78213 revisited (was Re: [PATCH 5/9] Introduce selftest::locate_file (v4))

2016-12-09 Thread David Malcolm
On Fri, 2016-12-09 at 02:48 +0100, Bernd Schmidt wrote: > On 12/08/2016 10:47 PM, David Malcolm wrote: > > > Is (A) OK, or would you prefer (B)? (I prefer (A) fwiw) > > Oh well, just keep it as it is. > > > Bernd Thanks. Unfortunately, applying the "locate_file" patch https://gcc.gnu.org/

Re: [PATCH 5/9] Introduce selftest::locate_file (v4)

2016-12-08 Thread Bernd Schmidt
On 12/08/2016 10:47 PM, David Malcolm wrote: Is (A) OK, or would you prefer (B)? (I prefer (A) fwiw) Oh well, just keep it as it is. Bernd

Re: [PATCH 5/9] Introduce selftest::locate_file (v4)

2016-12-08 Thread David Malcolm
On Thu, 2016-12-01 at 14:29 +0100, Bernd Schmidt wrote: > On 11/11/2016 10:15 PM, David Malcolm wrote: > > + /* Makefile.in has -fself-test=$(srcdir)/testsuite/selftests, so > > that > > + flag_self_test contains the path to the selftest subdirectory > > of the > > + source tree (without a

Re: [PATCH 5/9] Introduce selftest::locate_file (v4)

2016-12-01 Thread David Malcolm
On Thu, 2016-12-01 at 14:29 +0100, Bernd Schmidt wrote: > On 11/11/2016 10:15 PM, David Malcolm wrote: > > + /* Makefile.in has -fself-test=$(srcdir)/testsuite/selftests, so > > that > > + flag_self_test contains the path to the selftest subdirectory > > of the > > + source tree (without a

Re: [PATCH 5/9] Introduce selftest::locate_file (v4)

2016-12-01 Thread Bernd Schmidt
On 11/11/2016 10:15 PM, David Malcolm wrote: + /* Makefile.in has -fself-test=$(srcdir)/testsuite/selftests, so that + flag_self_test contains the path to the selftest subdirectory of the + source tree (without a trailing slash). Copy it up to + path_to_selftest_files, to avoid self

[PATCH 5/9] Introduce selftest::locate_file (v4)

2016-11-11 Thread David Malcolm
Link to discussion of earlier version of patch: https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00260.html On Wed, 2016-10-05 at 18:10 +0200, Bernd Schmidt wrote: > On 10/05/2016 06:15 PM, David Malcolm wrote: > > selftest: s-selftest > > s-selftest: $(GCC_PASSES) cc1$(exeext) stmp-int-hdrs > >