Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-21 Thread John Paul Adrian Glaubitz
Hi Bernd! This should work: # binary default deb http://ftp.ports.debian.org/debian-ports/ unstable main deb http://incoming.ports.debian.org/buildd/ unstable main deb http://ftp.ports.debian.org/debian-ports/ unreleased main # source deb-src http://ftp.debian.org/debian/ unstable main deb-src

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-21 Thread Bernd Schmidt
On 11/21/19 1:30 PM, Matthias Klose wrote: > > that would be apt build-dep gcc-9. The former would only install the build > dependencies of the gcc-defaults package. That gets me E: You must put some 'source' URIs in your sources.list where /etc/apt/sources.list looks like deb

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-21 Thread Matthias Klose
On 20.11.19 22:38, Richard Earnshaw wrote: > On 20/11/2019 20:48, Bernd Schmidt wrote: >> On 11/20/19 8:27 PM, Mikael Pettersson wrote: >>> On Wed, Nov 20, 2019 at 3:16 PM Bernd Schmidt >>> wrote: Probably best to just run tests on stage1 and hope something shows up. >>> >>> Ok, how do I

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Richard Earnshaw
On 20/11/2019 20:48, Bernd Schmidt wrote: > On 11/20/19 8:27 PM, Mikael Pettersson wrote: >> On Wed, Nov 20, 2019 at 3:16 PM Bernd Schmidt wrote: >>> Probably best to just run tests on stage1 and hope something shows up. >> >> Ok, how do I did that? I've always just done 'make -k check' after >>

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Bernd Schmidt
On 11/20/19 8:27 PM, Mikael Pettersson wrote: > On Wed, Nov 20, 2019 at 3:16 PM Bernd Schmidt wrote: >> Probably best to just run tests on stage1 and hope something shows up. > > Ok, how do I did that? I've always just done 'make -k check' after > full bootstraps. > I assume the stage 1

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Jeff Law
On 11/20/19 12:27 PM, Mikael Pettersson wrote: > On Wed, Nov 20, 2019 at 3:16 PM Bernd Schmidt > wrote: >> >> On 11/20/19 2:50 PM, Mikael Pettersson wrote: >>> On Mon, Nov 18, 2019 at 9:57 PM Mikael Pettersson >>> wrote: On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt wrote:

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Mikael Pettersson
On Wed, Nov 20, 2019 at 3:16 PM Bernd Schmidt wrote: > > On 11/20/19 2:50 PM, Mikael Pettersson wrote: > > On Mon, Nov 18, 2019 at 9:57 PM Mikael Pettersson > > wrote: > >> > >> On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt > >> wrote: > >>> > >>> Hi Mikael, > >>> > This fixed the

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Jeff Law
On 11/20/19 7:16 AM, Bernd Schmidt wrote: > On 11/20/19 2:50 PM, Mikael Pettersson wrote: >> On Mon, Nov 18, 2019 at 9:57 PM Mikael Pettersson >> wrote: >>> >>> On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt >>> wrote: Hi Mikael, > This fixed the problem, thanks.

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Bernd Schmidt
On 11/20/19 2:50 PM, Mikael Pettersson wrote: > On Mon, Nov 18, 2019 at 9:57 PM Mikael Pettersson > wrote: >> >> On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt wrote: >>> >>> Hi Mikael, >>> This fixed the problem, thanks. >>> >>> Could you also run the testsuite to see if you can reproduce

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-20 Thread Mikael Pettersson
On Mon, Nov 18, 2019 at 9:57 PM Mikael Pettersson wrote: > > On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt wrote: > > > > Hi Mikael, > > > > > This fixed the problem, thanks. > > > > Could you also run the testsuite to see if you can reproduce the > > g++.old-deja failures Andreas reported? > >

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-18 Thread Mikael Pettersson
On Mon, Nov 18, 2019 at 8:31 PM Bernd Schmidt wrote: > > Hi Mikael, > > > This fixed the problem, thanks. > > Could you also run the testsuite to see if you can reproduce the > g++.old-deja failures Andreas reported? Yes, but it will probably take another week before the native bootstrap (on

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-18 Thread Bernd Schmidt
Hi Mikael, > This fixed the problem, thanks. Could you also run the testsuite to see if you can reproduce the g++.old-deja failures Andreas reported? Bernd

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-17 Thread Mikael Pettersson
On Sun, Nov 17, 2019 at 5:57 PM Andreas Schwab wrote: > > On Nov 17 2019, Mikael Pettersson wrote: > > > /tmp/ccJA1qws.s:4828: Error: operands mismatch -- statement `seq %a1' > > ignored > > /tmp/ccJA1qws.s:7344: Error: operands mismatch -- statement `seq %a1' > > ignored > > That should fix

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-17 Thread Jeff Law
On 11/17/19 9:57 AM, Andreas Schwab wrote: > On Nov 17 2019, Mikael Pettersson wrote: > >> /tmp/ccJA1qws.s:4828: Error: operands mismatch -- statement `seq %a1' ignored >> /tmp/ccJA1qws.s:7344: Error: operands mismatch -- statement `seq %a1' ignored > > That should fix it: > > diff --git

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-17 Thread Andreas Schwab
On Nov 17 2019, Mikael Pettersson wrote: > /tmp/ccJA1qws.s:4828: Error: operands mismatch -- statement `seq %a1' ignored > /tmp/ccJA1qws.s:7344: Error: operands mismatch -- statement `seq %a1' ignored That should fix it: diff --git a/gcc/config/m68k/m68k.md b/gcc/config/m68k/m68k.md index

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-17 Thread Mikael Pettersson
On Wed, 13 Nov 2019 14:04:59 +0100, Bernd Schmidt wrote: > This is a set of patches to convert m68k so that it no longer uses cc0. Thank you for doing this. I attempted a native bootstrap of gcc-10-20191110 (r278028) plus the five patches posted so far on m68k-linux (aranym), but it failed in

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-16 Thread Bernd Schmidt
On 11/16/19 9:18 AM, Andreas Schwab wrote: > On Nov 16 2019, Bernd Schmidt wrote: > >> Well, there has to be some difference between what you are doing and >> what I am doing, because: >> >> Running /local/src/egcs/git/gcc/testsuite/g++.old-deja/old-deja.exp ... >> >> === g++ Summary

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-16 Thread John Paul Adrian Glaubitz
Hi Andreas! > === g++ Summary === > > # of expected passes26803 > # of unexpected failures16 > # of expected failures 82 > # of unsupported tests 157 > /daten/aranym/gcc/gcc-20191116/Build/gcc/xg++ version 10.0.0 20191115 > (experimental)

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-16 Thread Andreas Schwab
Running /daten/aranym/gcc/gcc-20191116/gcc/testsuite/g++.old-deja/old-deja.exp ... FAIL: g++.old-deja/g++.other/dyncast1.C -std=c++98 execution test FAIL: g++.old-deja/g++.other/dyncast1.C -std=c++14 execution test FAIL: g++.old-deja/g++.other/dyncast1.C -std=c++17 execution test FAIL:

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-16 Thread Andreas Schwab
On Nov 16 2019, Bernd Schmidt wrote: > Well, there has to be some difference between what you are doing and > what I am doing, because: > > Running /local/src/egcs/git/gcc/testsuite/g++.old-deja/old-deja.exp ... > > === g++ Summary === > > # of expected passes 26826 > # of

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Bernd Schmidt
On 11/15/19 11:50 PM, Andreas Schwab wrote: > On Nov 15 2019, Bernd Schmidt wrote: > >> I meant the compiler command line of course... for any -mcpu flags that >> might differ from my test run. > > There are none. Well, there has to be some difference between what you are doing and what I am

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Andreas Schwab
On Nov 15 2019, Bernd Schmidt wrote: > I meant the compiler command line of course... for any -mcpu flags that > might differ from my test run. There are none. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Bernd Schmidt
On 11/15/19 10:58 PM, Andreas Schwab wrote: > On Nov 15 2019, Bernd Schmidt wrote: > >> Any chance you could show the command lines from the log files or some >> other way of reproducing the issue? > > Executing on aranym: OMP_NUM_THREADS=2 >

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Andreas Schwab
On Nov 15 2019, Bernd Schmidt wrote: > Any chance you could show the command lines from the log files or some > other way of reproducing the issue? Executing on aranym: OMP_NUM_THREADS=2

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Bernd Schmidt
On 11/15/19 5:34 PM, Andreas Schwab wrote: > On Nov 15 2019, Bernd Schmidt wrote: > >> Are these with the patch? > > Yes. > >> Are you on real hardware > > No, I'm using aranym. Any chance you could show the command lines from the log files or some other way of reproducing the issue? Bernd

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Andreas Schwab
On Nov 15 2019, Bernd Schmidt wrote: > Are these with the patch? Yes. > Are you on real hardware No, I'm using aranym. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now for something completely different."

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Bernd Schmidt
On 11/15/19 2:48 PM, Andreas Schwab wrote: > Here are the results of running the testsuite on m68k-linux: > > http://gcc.gnu.org/ml/gcc-testresults/2019-11/msg00908.html > > This is a list of regressions: Are these with the patch? I'm not seeing any of these in my testing with qemu. Are you on

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread John Paul Adrian Glaubitz
Hi! On 11/15/19 2:45 PM, John Paul Adrian Glaubitz wrote: >> It works on the kernel build, at least. No idea if this *runs*, I just >> built it ;-) > > I just did that and kernel 5.3.9 built with gcc trunk with Bernd's patches > boots fine on qemu-m68k-system. I will test the kernel on a real

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread Andreas Schwab
Here are the results of running the testsuite on m68k-linux: http://gcc.gnu.org/ml/gcc-testresults/2019-11/msg00908.html This is a list of regressions: g++.old-deja/g++.other/dyncast1.C -std=c++14 execution test g++.old-deja/g++.other/dyncast1.C -std=c++17 execution test

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-15 Thread John Paul Adrian Glaubitz
Hi Segher! > It works on the kernel build, at least. No idea if this *runs*, I just > built it ;-) I just did that and kernel 5.3.9 built with gcc trunk with Bernd's patches boots fine on qemu-m68k-system. I will test the kernel on a real Amiga later but I'm confident it will work there as

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-14 Thread Richard Henderson
On 11/13/19 8:35 PM, Jeff Law wrote: > On 11/13/19 6:04 AM, Bernd Schmidt wrote: >> The cc0 machinery allows for eliminating unnecessary comparisons by >> examining the effect instructions have on the flags registers. I have >> replicated that mechanism with a relatively modest amount of code

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-13 Thread Jeff Law
On 11/13/19 6:04 AM, Bernd Schmidt wrote: > This is a set of patches to convert m68k so that it no longer uses cc0. > The approach is to combine cc0 setter/user pairs into cbranch and cstore > patterns. It does not expose the flag register directly. Since m68k is a > target that is not under

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-13 Thread Segher Boessenkool
On Wed, Nov 13, 2019 at 07:57:58PM +0100, Bernd Schmidt wrote: > On 11/13/19 7:16 PM, Segher Boessenkool wrote: > > I tried this out with a kernel build (just the defconfig). > > > during RTL pass: jump2 > > /home/segher/src/kernel/fs/binfmt_elf.c: In function 'elf_core_dump': > >

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-13 Thread Bernd Schmidt
On 11/13/19 7:16 PM, Segher Boessenkool wrote: > I tried this out with a kernel build (just the defconfig). > during RTL pass: jump2 > /home/segher/src/kernel/fs/binfmt_elf.c: In function 'elf_core_dump': > /home/segher/src/kernel/fs/binfmt_elf.c:2409:1: internal compiler error: in >

Re: [PATCH 0/4] Eliminate cc0 from m68k

2019-11-13 Thread Segher Boessenkool
On Wed, Nov 13, 2019 at 02:04:59PM +0100, Bernd Schmidt wrote: > This is a set of patches to convert m68k so that it no longer uses cc0. I tried this out with a kernel build (just the defconfig). First problem was patch 4 doesn't apply, it has white-space damage. It's small, I fixed that up