Re: [PATCH v4] Run tests only if the machine supports the instruction set.

2016-12-22 Thread Andreas Krebbel
> gcc/ChangeLog-archlevel > > * config/s390/s390-c.c (s390_cpu_cpp_builtins_internal): Define > __S390_ARCH_LEVEL__. > gcc/testsuite/ChangeLog-setmem > > * gcc.target/s390/md/setmem_long-1.c: Use "s390_useable_hw". > * gcc.target/s390/md/rXsbg_mode_sXl.c: Likewise. >

Re: [PATCH v4] Run tests only if the machine supports the instruction set.

2016-12-21 Thread Dominik Vogt
On Tue, Dec 20, 2016 at 10:32:26PM +0100, Jakub Jelinek wrote: > On Tue, Dec 20, 2016 at 10:26:13PM +0100, Dominik Vogt wrote: > > On Tue, Dec 20, 2016 at 11:57:52AM -0800, Mike Stump wrote: > > > On Dec 20, 2016, at 6:10 AM, Dominik Vogt wrote: > > > > Right, it gets

Re: [PATCH v4] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Jakub Jelinek
On Tue, Dec 20, 2016 at 10:26:13PM +0100, Dominik Vogt wrote: > On Tue, Dec 20, 2016 at 11:57:52AM -0800, Mike Stump wrote: > > On Dec 20, 2016, at 6:10 AM, Dominik Vogt wrote: > > > Right, it gets called even more often than one would think, and > > > even with empty

Re: [PATCH v4] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Dominik Vogt
On Tue, Dec 20, 2016 at 11:57:52AM -0800, Mike Stump wrote: > On Dec 20, 2016, at 6:10 AM, Dominik Vogt wrote: > > Right, it gets called even more often than one would think, and > > even with empty torture_current_options. The attached new patch > > (v3) removes -Ox

Re: [PATCH v3] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Mike Stump
On Dec 20, 2016, at 6:10 AM, Dominik Vogt wrote: > Right, it gets called even more often than one would think, and > even with empty torture_current_options. The attached new patch > (v3) removes -Ox options and superflous whitespace and caches that > between calls if

Re: [PATCH v3] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Dominik Vogt
On Tue, Dec 20, 2016 at 11:32:58AM +0100, Jakub Jelinek wrote: > On Tue, Dec 20, 2016 at 11:22:47AM +0100, Dominik Vogt wrote: > > On Mon, Dec 19, 2016 at 06:00:21PM +0100, Jakub Jelinek wrote: > > > On Mon, Dec 19, 2016 at 05:50:40PM +0100, Dominik Vogt wrote: > > > > *

Re: [PATCH v2] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Jakub Jelinek
On Tue, Dec 20, 2016 at 11:22:47AM +0100, Dominik Vogt wrote: > On Mon, Dec 19, 2016 at 06:00:21PM +0100, Jakub Jelinek wrote: > > On Mon, Dec 19, 2016 at 05:50:40PM +0100, Dominik Vogt wrote: > > > * config/s390/s390-c.c (s390_cpu_cpp_builtins_internal): Define > > > __S390_ARCH_LEVEL__. > >

Re: [PATCH v2] Run tests only if the machine supports the instruction set.

2016-12-20 Thread Dominik Vogt
On Mon, Dec 19, 2016 at 06:00:21PM +0100, Jakub Jelinek wrote: > On Mon, Dec 19, 2016 at 05:50:40PM +0100, Dominik Vogt wrote: > > * config/s390/s390-c.c (s390_cpu_cpp_builtins_internal): Define > > __S390_ARCH_LEVEL__. > > gcc/testsuite/ChangeLog-setmem > > > > *

Re: [PATCH v2] Run tests only if the machine supports the instruction set.

2016-12-19 Thread Jakub Jelinek
On Mon, Dec 19, 2016 at 05:50:40PM +0100, Dominik Vogt wrote: > * config/s390/s390-c.c (s390_cpu_cpp_builtins_internal): Define > __S390_ARCH_LEVEL__. > gcc/testsuite/ChangeLog-setmem > > * gcc.target/s390/md/setmem_long-1.c: Use "runnable". > *

Re: [PATCH v2] Run tests only if the machine supports the instruction set.

2016-12-19 Thread Dominik Vogt
On Mon, Dec 19, 2016 at 03:28:06PM +0100, Dominik Vogt wrote: > The attached patch is specific to S/390 but contains a small > common code change in gcc-dg.exp. It fixes the notorious problem > of md tests running on an S/390 machine that does not support the > z13 instruction set. > >

Run tests only if the machine supports the instruction set.

2016-12-19 Thread Dominik Vogt
The attached patch is specific to S/390 but contains a small common code change in gcc-dg.exp. It fixes the notorious problem of md tests running on an S/390 machine that does not support the z13 instruction set. Bootstrapped and tested on s390x biarch. Ciao Dominik ^_^ ^_^ -- Dominik Vogt