RE: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-20 Thread Taylor Simpson
inaro.org; Brian Cain > ; Matheus Bernardino (QUIC) > ; stefa...@redhat.com > Subject: Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef- > parser > > On 20/12/22 13:51, Alessandro Di Federico wrote: > > On Tue, 20 Dec 2022 08:30:02 +0100 > > Philippe Mathieu-Daudé

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-20 Thread Philippe Mathieu-Daudé
On 20/12/22 13:51, Alessandro Di Federico wrote: On Tue, 20 Dec 2022 08:30:02 +0100 Philippe Mathieu-Daudé wrote: Do we really need this level? IIUC the problem with -Wextra is using a newer compiler toolchain it can include warnings we haven't fixed. Maybe worthwhile but it can break from tim

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-20 Thread Alessandro Di Federico via
On Tue, 20 Dec 2022 08:30:02 +0100 Philippe Mathieu-Daudé wrote: > Do we really need this level? IIUC the problem with -Wextra is using a > newer compiler toolchain it can include warnings we haven't fixed. > Maybe worthwhile but it can break from times to times. I think we just wanted to be ove

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-20 Thread Alessandro Di Federico via
On Mon, 19 Dec 2022 18:54:22 + Taylor Simpson wrote: > > Applied, thanks. > > Thanks!! Thanks from our side too! :) We started this project back in 2015, I'm really happy we finally got it in! -- Alessandro Di Federico rev.ng Labs

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-19 Thread Philippe Mathieu-Daudé
On 20/12/22 00:19, Philippe Mathieu-Daudé wrote: Hi, On 16/12/22 21:48, Taylor Simpson wrote: 12-21) Instruction definition parser (idef-parser) from rev.ng Parses the instruction semantics and generates TCG Building QEMU wi

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-19 Thread Philippe Mathieu-Daudé
Hi, On 16/12/22 21:48, Taylor Simpson wrote: 12-21) Instruction definition parser (idef-parser) from rev.ng Parses the instruction semantics and generates TCG Building QEMU with Clang I'm now getting: target/hexagon/idef-par

RE: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-19 Thread Taylor Simpson
Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef- > parser > > On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: > > > > The following changes since commit > 4208e6ae114ac8266dcacc9696a443ce5c37b04e: > > > > Merge tag 'pull-request-2022-12-

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-19 Thread Philippe Mathieu-Daudé
On 18/12/22 18:01, Peter Maydell wrote: On Sun, 18 Dec 2022 at 16:53, Richard Henderson wrote: On 12/18/22 05:52, Peter Maydell wrote: On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: Merge tag 'pull-reque

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-19 Thread Peter Maydell
On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: > > The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: > > Merge tag 'pull-request-2022-12-15' of https://gitlab.com/thuth/qemu into > staging (2022-12-15 21:39:56 +) > > are available in the Git repository at: >

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-18 Thread Peter Maydell
On Sun, 18 Dec 2022 at 16:53, Richard Henderson wrote: > > On 12/18/22 05:52, Peter Maydell wrote: > > On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: > >> > >> The following changes since commit > >> 4208e6ae114ac8266dcacc9696a443ce5c37b04e: > >> > >>Merge tag 'pull-request-2022-12-15'

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-18 Thread Richard Henderson
On 12/18/22 05:52, Peter Maydell wrote: On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: Merge tag 'pull-request-2022-12-15' of https://gitlab.com/thuth/qemu into staging (2022-12-15 21:39:56 +) are avail

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-18 Thread Anton Johansson via
On 12/18/22 14:52, Peter Maydell wrote: Failed to build, s390x: https://gitlab.com/qemu-project/qemu/-/jobs/3492490152 Program scripts/decodetree.py found: YES (/usr/bin/python3 /home/gitlab-runner/builds/-LCfcJ2T/0/qemu-project/qemu/scripts/decodetree.py) Program flex found: NO ../target/hexa

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-18 Thread Peter Maydell
On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: > > The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: > > Merge tag 'pull-request-2022-12-15' of https://gitlab.com/thuth/qemu into > staging (2022-12-15 21:39:56 +) > > are available in the Git repository at: >

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-18 Thread Anton Johansson via
On 12/17/22 22:21, Peter Maydell wrote: Does that mean somebody needs to do something before I can merge this? thanks -- PMM Hi Peter, no one needs to do anything about the CI, it has already been taken care of. All containers that need flex/bison have it, and the debian-hexagon-cross cont

Re: [PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-17 Thread Peter Maydell
On Fri, 16 Dec 2022 at 20:49, Taylor Simpson wrote: > > The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: > > Merge tag 'pull-request-2022-12-15' of https://gitlab.com/thuth/qemu into > staging (2022-12-15 21:39:56 +) > > are available in the Git repository at: >

[PULL 00/21] Hexagon update: bug fixes, performance, idef-parser

2022-12-16 Thread Taylor Simpson
The following changes since commit 4208e6ae114ac8266dcacc9696a443ce5c37b04e: Merge tag 'pull-request-2022-12-15' of https://gitlab.com/thuth/qemu into staging (2022-12-15 21:39:56 +) are available in the Git repository at: https://github.com/quic/qemu tags/pull-hex-20221216-1 for you t