Re: [PATCH v6 34/35] Auto-import Docker support files

On Tue, 12 Jan 2021 11:58:30 + Alex Bennée wrote: > > -for filename in args.extra_files or []: > > +extra_files = args.extra_files or [] > > +extra_files += glob(basename + ".*") > > +for filename in extra_files: > > Hmm not so sure about

Re: [RFC PATCH v5 20/33] Hexagon (target/hexagon) generator phase 2 - generate header files

On Thu, 29 Oct 2020 19:08:26 -0500 Taylor Simpson wrote: > +def genptr_decl_new(f,regtype,regid,regno): > +if (regtype == "N"): > +if (regid in {"s", "t"}): > +f.write("const int %s%sX = insn->regno[%d];\n" % \ > +(regtype, regid, regno)) > +

Re: [RFC PATCH v5 20/33] Hexagon (target/hexagon) generator phase 2 - generate header files

On Thu, 29 Oct 2020 19:08:26 -0500 Taylor Simpson wrote: > +from hex_common import * I'd suggest to avoid `import *`. See: python -c 'import this' | sed -n '4p' -- Alessandro Di Federico rev.ng

Re: [RFC PATCH v5 22/33] Hexagon (target/hexagon) generater phase 4 - decode tree

On Thu, 29 Oct 2020 19:08:28 -0500 Taylor Simpson wrote: > +if __name__ == '__main__': > +f = io.StringIO() > +print_tree(f, dectree_normal) > +print_tree(f, dectree_16bit) > +if subinsn_groupings: > +print_tree(f, dectree_subinsn_groupings) > +for (name,

[PATCH v5 11/14] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_helper_funcs.py | 5 ++- target/hexagon/gen_helper_protos.py | 5 ++-

[PATCH v5 02/14] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico Reviewed-by: Richard Henderson --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7d9cd29042..7fd32062be 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -192,6 +192,8 @@

[PATCH v5 09/14] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.h | 262 target/hexagon/idef-parser/idef-parser.lex| 597 ++ target/hexagon/meson.build| 4 +

[PATCH v5 00/14] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It's the fifth iteration of the patchset and includes fixes suggested in previous iterations. This patchset also drops some macro overrides that are no longer necessary, since the instructions they were

[PATCH v5 04/14] target/hexagon: make slot number an unsigned

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel Acked-by: Richard Henderson --- target/hexagon/genptr.c | 6 -- target/hexagon/macros.h | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/target/hexagon/genptr.c

[PATCH v5 08/14] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 ++ target/hexagon/idef-parser/macros.inc | 153

[PATCH v5 06/14] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- target/hexagon/genptr.c | 163 target/hexagon/genptr.h | 23 ++ target/hexagon/macros.h |

[PATCH v5 01/14] tcg: expose TCGCond manipulation routines

From: Alessandro Di Federico This commit moves into a separate file routines used to manipulate TCGCond. These will be employed by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- include/tcg/tcg-cond.h | 101 +

[PATCH v5 07/14] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel Reviewed-by: Richard Henderson --- target/hexagon/translate.c | 3 ++- target/hexagon/translate.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/target/hexagon/translate.c

[PATCH v5 13/14] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- tests/tcg/hexagon/Makefile.target | 36 - tests/tcg/hexagon/crt.S| 28 + tests/tcg/hexagon/test_abs.S | 20 ++ tests/tcg/hexagon/test_add.S |

[PATCH v5 10/14] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.y | 961 +++ target/hexagon/idef-parser/parser-helpers.c | 2396 + target/hexagon/idef-parser/parser-helpers.h | 347 +++

[PATCH v5 05/14] target/hexagon: make helper functions non-static

From: Paolo Montesel Make certain helper functions non-static, making them available outside genptr.c. These functions are required by code generated by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel Reviewed-by: Richard Henderson ---

[PATCH v5 12/14] target/hexagon: remove unused macros and functions

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/gen_tcg.h | 528 - target/hexagon/genptr.c| 98 --- target/hexagon/macros.h| 200 +- target/hexagon/op_helper.c | 119

[PATCH v5 03/14] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/README | 5 + target/hexagon/idef-parser/README.rst | 447 ++ 2 files changed, 452 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[PATCH v5 14/14] gitlab-ci: do not use qemu-project Docker registry

From: Alessandro Di Federico This commit is necessary in order to use container built by the current run of the CI. If we don't do this, we use official containers which are not affected by the additional dependencies we're introducing. Signed-off-by: Alessandro Di Federico ---

Re: [PATCH v5 03/14] target/hexagon: import README for idef-parser

On Wed, 23 Jun 2021 15:46:22 + Taylor Simpson wrote: > The output isn't actually indented, but it would be great if it were. > This is especially true for instructions where an "if" or "for" show > up in the emitted code. Emitting whitespaces in the parser is a bit annoying and fragile,

Re: [PATCH v5 14/14] gitlab-ci: do not use qemu-project Docker registry

On Tue, 29 Jun 2021 15:37:21 +0100 Daniel P. Berrangé wrote: > We have either one or two images. If this is a new user fork, > we only have the $COMMON_TAG image in the main QEMU registry. > If this is a existing fork, we might have two images, one in > the main QEMU registry and one in the

Re: [PATCH v5 10/14] target/hexagon: import parser for idef-parser

On Thu, 24 Jun 2021 03:55:35 + Taylor Simpson wrote: > > + | rvalue '[' rvalue ']' > > + { > > + @1.last_column = @4.last_column; > > + if ($3.type == IMMEDIATE) { > > + $$ = gen_tmp(c, &@1, $1.bit_width); > > + OUT(c,

Re: [PATCH v5 14/14] gitlab-ci: do not use qemu-project Docker registry

On Sat, 19 Jun 2021 11:37:13 +0200 Alessandro Di Federico wrote: > From: Alessandro Di Federico > > This commit is necessary in order to use container built by the > current run of the CI. If we don't do this, we use official > containers which are not affected by the additional dependencies >

Re: [PATCH v5 10/14] target/hexagon: import parser for idef-parser

On Thu, 24 Jun 2021 03:55:35 + Taylor Simpson wrote: > > +void gen_deposit_op(Context *c, > > +YYLTYPE *locp, > > +HexValue *dest, > > +HexValue *value, > > +HexValue *index, > > +HexCast

Re: [PATCH v5 13/14] target/hexagon: import additional tests

On Fri, 25 Jun 2021 18:56:57 -0500 Taylor Simpson wrote: > Each of these are very small, so I recommend putting them into misc.c > or combine all the assembly into a small number of executables. These tests are designed to run without libc and test in isolation very specific functionality

[PATCH v3 04/12] target/hexagon: make slot number an unsigned

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/genptr.c | 6 -- target/hexagon/macros.h | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/target/hexagon/genptr.c b/target/hexagon/genptr.c index

[PATCH v3 09/12] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.h | 240 +++ target/hexagon/idef-parser/idef-parser.lex| 611 ++ target/hexagon/meson.build| 4 +

[PATCH v3 10/12] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.y | 940 +++ target/hexagon/idef-parser/parser-helpers.c | 2230 + target/hexagon/idef-parser/parser-helpers.h | 344 +++

[PATCH v3 03/12] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/README | 5 + target/hexagon/idef-parser/README.rst | 447 ++ 2 files changed, 452 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[PATCH v3 07/12] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/translate.c | 3 ++- target/hexagon/translate.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/target/hexagon/translate.c b/target/hexagon/translate.c index

[PATCH v3 05/12] target/hexagon: make helper functions non-static

From: Paolo Montesel Make certain helper functions non-static, making them available outside genptr.c. These functions are required by code generated by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/genptr.c | 7 ---

[PATCH v3 01/12] tcg: expose TCGCond manipulation routines

From: Alessandro Di Federico This commit moves into a separate file routines used to manipulate TCGCond. These will be employed by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- include/tcg/tcg-cond.h | 101 +

[PATCH v3 08/12] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 ++ target/hexagon/idef-parser/macros.inc | 150

[PATCH v3 00/12] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It's the third iteration of the patchset and includes fixes suggested in the previous iteration. `idef-parser` is a build-time tool built using flex and bison. Its aim is to generate a large part of the

[PATCH v3 02/12] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 554be84b32..13de7ecc36 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -194,11 +194,19 @@ Hexagon TCG CPUs M: Taylor

[PATCH v3 11/12] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_tcg_funcs.py | 28 ++-- target/hexagon/hex_common.py| 10 ++

[PATCH v3 06/12] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- target/hexagon/genptr.c | 173 target/hexagon/genptr.h | 17 target/hexagon/macros.h | 9

[PATCH v3 12/12] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- tests/tcg/hexagon/Makefile.target | 35 - tests/tcg/hexagon/crt.S| 28 + tests/tcg/hexagon/test_abs.S | 20 ++ tests/tcg/hexagon/test_add.S |

Re: [PATCH v2 02/10] target/hexagon: import README for idef-parser

On Wed, 10 Mar 2021 15:48:14 + Taylor Simpson wrote: > Which instructions require this? There must be an attribute that we > could check to see if it is needed before emitting the TCG. The following should be an example of an instruction that requires zero-initialization: /*

[PATCH v2 06/10] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 target/hexagon/idef-parser/macros.inc | 166

[PATCH v2 07/10] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.h | 245 +++ target/hexagon/idef-parser/idef-parser.lex| 647 ++ target/hexagon/meson.build| 4 +

Re: [RFC PATCH 00/10] target/hexagon: introduce idef-parser

On Sun, 14 Feb 2021 13:52:25 -0800 Richard Henderson wrote: > Well, first off, this fails to build with our gitlab CI. > > There are several problems, apart from the new bison parse.error > option previously reported: > > (1) bison and flex not added to dockerfiles, so the packages are not >

[PATCH v2 02/10] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/README | 5 + target/hexagon/idef-parser/README.rst | 447 ++ 2 files changed, 452 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[PATCH v2 08/10] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.y | 1250 +++ target/hexagon/idef-parser/parser-helpers.c | 1925 + target/hexagon/idef-parser/parser-helpers.h | 293 +++

Re: [RFC PATCH 08/10] target/hexagon: import parser for idef-parser

On Sun, 14 Feb 2021 08:56:11 -0800 Richard Henderson wrote: > What version of bison are you using? We were using a rather recent version of bison (3.7.5), but it's not strictly required. Fixed in v2. -- Alessandro Di Federico rev.ng

[PATCH v2 00/10] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It's the second iteration of the patchset and includes fixes suggested in the previous iteration. `idef-parser` is a build-time tool built using flex and bison. Its aim is to generate a large part of the

[PATCH v2 03/10] target/hexagon: make helper functions non-static

From: Paolo Montesel Make certain helper functions non-static, making them available outside genptr.c. These functions are required by code generated by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/genptr.c | 13 ++---

[PATCH v2 01/10] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9b2aa18e1f..9a6878b970 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -192,11 +192,19 @@ Hexagon TCG CPUs M: Taylor

[PATCH v2 10/10] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- tests/tcg/hexagon/Makefile.target | 35 - tests/tcg/hexagon/crt.S| 28 + tests/tcg/hexagon/first.S | 24 ++-- tests/tcg/hexagon/test_abs.S |

[PATCH v2 04/10] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- target/hexagon/genptr.c | 227 +++- target/hexagon/genptr.h | 19 target/hexagon/macros.h | 2

[PATCH v2 09/10] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_tcg_funcs.py | 28 ++-- target/hexagon/hex_common.py| 10 ++

[PATCH v2 05/10] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/translate.c | 4 +++- target/hexagon/translate.h | 1 + 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/target/hexagon/translate.c b/target/hexagon/translate.c index

Re: [PATCH v2 09/10] target/hexagon: call idef-parser functions

On Thu, 25 Feb 2021 19:47:04 -0800 Richard Henderson wrote: > On 2/25/21 7:18 AM, Alessandro Di Federico wrote: > > +elif hex_common.is_new_val(regtype, regid, tag): > > +declared.append("%s%sN" % (regtype,regid)) > > +else: > > +print("Bad

Re: [PATCH v2 08/10] target/hexagon: import parser for idef-parser

On Thu, 25 Feb 2021 19:30:14 -0800 Richard Henderson wrote: > > +} > > +} > > +code > > +{ > > +if (c->inst.error_count != 0) { > > +fprintf(stderr, > > +"Parsing of instruction %s generated %d errors!\n", > > +c->inst.name, > > +

Re: [PATCH v2 02/10] target/hexagon: import README for idef-parser

On Thu, 25 Feb 2021 12:20:53 -0800 Richard Henderson wrote: > This is odd, as is the description of why. Yes, if RdV is read > without initialization, TCG middle-end will abort (at least with > --enable-debug-tcg enabling the assertions). But you've just said > that "no reading" was found.

Re: [PATCH 0/4] hexagon: Add Docker image & testing to gitlab-ci

On Sun, 28 Feb 2021 23:23:10 +0100 Philippe Mathieu-Daudé wrote: > This series is a rework of the 'Add Dockerfile for hexagon' patch > from Alessandro/Brian that Taylor sent in v8: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg780330.html > but adapted to mainstream. All of this looks

[PATCH v4 00/12] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It's the fourth iteration of the patchset and includes fixes suggested in previous iterations. `idef-parser` is a build-time tool built using flex and bison. Its aim is to generate a large part of the

[PATCH v4 05/12] target/hexagon: make helper functions non-static

From: Paolo Montesel Make certain helper functions non-static, making them available outside genptr.c. These functions are required by code generated by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/genptr.c | 7 ---

[PATCH v4 01/12] tcg: expose TCGCond manipulation routines

From: Alessandro Di Federico This commit moves into a separate file routines used to manipulate TCGCond. These will be employed by the idef-parser. Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- include/tcg/tcg-cond.h | 101 +

[PATCH v4 07/12] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/translate.c | 3 ++- target/hexagon/translate.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/target/hexagon/translate.c b/target/hexagon/translate.c index

[PATCH v4 03/12] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/README | 5 + target/hexagon/idef-parser/README.rst | 447 ++ 2 files changed, 452 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[PATCH v4 06/12] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- target/hexagon/genptr.c | 188 target/hexagon/genptr.h | 22 + target/hexagon/macros.h |

[PATCH v4 12/12] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- tests/tcg/hexagon/Makefile.target | 36 - tests/tcg/hexagon/crt.S| 28 + tests/tcg/hexagon/test_abs.S | 20 ++ tests/tcg/hexagon/test_add.S |

[PATCH v4 08/12] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 ++ target/hexagon/idef-parser/macros.inc | 150

[PATCH v4 10/12] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.y | 947 +++ target/hexagon/idef-parser/parser-helpers.c | 2374 + target/hexagon/idef-parser/parser-helpers.h | 347 +++

[PATCH v4 09/12] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.h | 254 target/hexagon/idef-parser/idef-parser.lex| 611 ++ target/hexagon/meson.build| 4 +

[PATCH v4 02/12] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 36055f14c5..158badbd18 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -193,11 +193,19 @@ Hexagon TCG CPUs M: Taylor

[PATCH v4 04/12] target/hexagon: make slot number an unsigned

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/genptr.c | 6 -- target/hexagon/macros.h | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/target/hexagon/genptr.c b/target/hexagon/genptr.c index

[PATCH v4 11/12] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_tcg_funcs.py | 28 ++-- target/hexagon/hex_common.py| 10 ++

Re: [PATCH v4 06/12] target/hexagon: introduce new helper functions

On Mon, 19 Apr 2021 15:00:17 + Taylor Simpson wrote: > Once this patch series is merged, many load/store instructions will > have manual overrides. I can easily provide overrides for the > remainder. Then, we could skip them in the idef-parser. At a > minimum, you should skip the ones

Re: [PATCH v8 35/35] Add Dockerfile for hexagon

On Sun, 14 Feb 2021 19:50:41 +0100 Philippe Mathieu-Daudé wrote: > Hi Alessandro! Hi Philippe! > I had to add this to build your image with QEMU build system: > > -- >8 -- > > diff --git a/tests/docker/Makefile.include > b/tests/docker/Makefile.include index 93b29ad823e..ba73fb4b1f0 100644

[RFC PATCH 08/10] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico --- target/hexagon/idef-parser/idef-parser.y| 1248 target/hexagon/idef-parser/parser-helpers.c | 1925 +++ target/hexagon/idef-parser/parser-helpers.h | 293 +++ target/hexagon/meson.build

[RFC PATCH 04/10] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico --- target/hexagon/genptr.c | 224 target/hexagon/genptr.h | 19 2 files changed, 243 insertions(+) diff --git

[RFC PATCH 03/10] target/hexagon: make helper functions non-static

From: Paolo Montesel Move certain helper functions required by code generated by the idef-parser available outside genptr.c, moving them into macros.h. This patch also introduces the gen_cancel and gen_fbrev helper which will be used by idef-parser. Signed-off-by: Alessandro Di Federico ---

[RFC PATCH 06/10] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 target/hexagon/idef-parser/macros.inc | 166

[RFC PATCH 07/10] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico --- target/hexagon/idef-parser/idef-lexer.lex | 648 ++ target/hexagon/idef-parser/idef-parser.h | 245 target/hexagon/meson.build| 4 + 3 files changed, 897 insertions(+) create mode

[RFC PATCH 02/10] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/README | 5 + target/hexagon/idef-parser/README.rst | 446 ++ 2 files changed, 451 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[RFC PATCH 05/10] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico --- target/hexagon/translate.c | 4 +++- target/hexagon/translate.h | 1 + 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/target/hexagon/translate.c b/target/hexagon/translate.c index eeaad5f8ba..a59db485a3 100644 ---

[RFC PATCH 01/10] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 41300089b2..029f96d3bc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -192,11 +192,19 @@ Hexagon TCG CPUs M: Taylor

[RFC PATCH 00/10] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It is based on "[PATCH v8 00/35] Hexagon patch series". `idef-parser` is a build-time tool built using flex and bison. Its aim is to generate a large part of the tiny code generator frontend for Hexagon.

[RFC PATCH 09/10] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_tcg_funcs.py | 28 ++-- target/hexagon/hex_common.py |

[RFC PATCH 10/10] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico --- tests/tcg/hexagon/Makefile.target | 35 - tests/tcg/hexagon/crt.S| 28 + tests/tcg/hexagon/first.S | 24 ++-- tests/tcg/hexagon/test_abs.S | 20 ++

QEMU CI failure of cross-i386-* targets (meson picks wrong glib for native target)

Hi Paolo, I'm trying to get the QEMU CI run successfully for the idef-parser patchset. However I'm facing an issue I haven't been able to work around with meson. Maybe you can help? The failing tests are cross-i386-* https://gitlab.com/carl.cudig/qemu/-/jobs/1437392669

CI for qemu-hexagon

Hi Alex, I hope you enjoyed the holidays! We're trying to upstream idef-parser (the automatic generator of the Hexagon frontend). This introduces new dependencies, specifically flex and bison. Attached you can find our patch for that. However the CI fails:

Re: QEMU CI failure of cross-i386-* targets (meson picks wrong glib for native target)

On Fri, 24 Dec 2021 22:25:24 +0100 Paolo Bonzini wrote: > Is the configure script setting $cross_compile to yes? That will > decide whether meson getting a --cross-file or a --native-file > option, and consequently whether it treats the host and build > machines as equal or different. From what

[PATCH v6 01/12] target/hexagon: update MAINTAINERS for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico Reviewed-by: Richard Henderson --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 4256ad1adb..6c825a9eb4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -194,6 +194,8 @@

[PATCH v6 05/12] target/hexagon: introduce new helper functions

From: Niccolò Izzo These helpers will be employed by the idef-parser generated code. Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- target/hexagon/genptr.h | 15 +- target/hexagon/macros.h | 9 target/hexagon/genptr.c | 113

[PATCH v6 03/12] target/hexagon: make slot number an unsigned

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel Acked-by: Richard Henderson --- target/hexagon/macros.h | 2 +- target/hexagon/genptr.c | 24 +--- 2 files changed, 14 insertions(+), 12 deletions(-) diff --git

[PATCH v6 06/12] target/hexagon: expose next PC in DisasContext

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel Reviewed-by: Richard Henderson Reviewed-by: Taylor Simpson --- target/hexagon/translate.h | 1 + target/hexagon/translate.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH v6 10/12] target/hexagon: call idef-parser functions

From: Alessandro Di Federico Extend gen_tcg_funcs.py in order to emit calls to the functions emitted by the idef-parser, if available. Signed-off-by: Alessandro Di Federico Reviewed-by: Taylor Simpson --- target/hexagon/gen_helper_funcs.py | 5 +- target/hexagon/gen_helper_protos.py | 5

[PATCH v6 08/12] target/hexagon: import lexer for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/idef-parser.h | 258 +++ target/hexagon/idef-parser/idef-parser.lex| 642 ++ target/hexagon/meson.build| 4 +

[PATCH v6 12/12] gitlab-ci: do not use qemu-project Docker registry

From: Alessandro Di Federico This commit is necessary in order to use container built by the current run of the CI. If we don't do this, we use official containers which are not affected by the additional dependencies we're introducing. This should be considered as a temporary solution in order

[PATCH v6 07/12] target/hexagon: prepare input for the idef-parser

From: Alessandro Di Federico Introduce infrastructure necessary to produce a file suitable for being parsed by the idef-parser. Signed-off-by: Alessandro Di Federico --- target/hexagon/gen_idef_parser_funcs.py | 114 ++ target/hexagon/idef-parser/macros.inc | 153

[PATCH v6 04/12] target/hexagon: make helper functions non-static

From: Paolo Montesel Make certain helper functions non-static, making them available outside genptr.c. These functions are required by code generated by the idef-parser. This commit also makes some op_helper.c non-static in order to avoid having them marked as unused when using the idef-parser

[PATCH v6 11/12] target/hexagon: import additional tests

From: Niccolò Izzo Signed-off-by: Alessandro Di Federico Signed-off-by: Niccolò Izzo --- tests/tcg/hexagon/Makefile.target | 30 +- tests/tcg/hexagon/crt.S| 26 tests/tcg/hexagon/test_abs.S | 20 ++ tests/tcg/hexagon/test_bitcnt.S| 42

[PATCH v6 00/12] target/hexagon: introduce idef-parser

From: Alessandro Di Federico This patchset introduces the idef-parser for target/hexagon. It's the sixth iteration of the patchset and includes fixes suggested in previous iterations. `idef-parser` is a build-time tool built using flex and bison. Its aim is to generate a large part of the tiny

[PATCH v6 02/12] target/hexagon: import README for idef-parser

From: Alessandro Di Federico Signed-off-by: Alessandro Di Federico --- target/hexagon/idef-parser/README.rst | 447 ++ target/hexagon/README | 5 + 2 files changed, 452 insertions(+) create mode 100644 target/hexagon/idef-parser/README.rst diff --git

[PATCH v6 09/12] target/hexagon: import parser for idef-parser

From: Paolo Montesel Signed-off-by: Alessandro Di Federico Signed-off-by: Paolo Montesel --- target/hexagon/idef-parser/parser-helpers.h | 346 +++ target/hexagon/idef-parser/parser-helpers.c | 2396 + target/hexagon/idef-parser/idef-parser.y | 975 +++

Re: [PATCH v6 0/7] tests: Refresh lcitool submodule & remove libxml2

On Tue, 25 Jan 2022 11:59:38 +0100 Philippe Mathieu-Daudé via wrote: > I'm seeing the same issue with these domains since mid december: > > ... > - rev.ng > > ... > https://lore.kernel.org/qemu-devel/20220105185720.0d4fc159@orange/ > ... I've tried to look into this and it looks like our set

Re: Any interest in a QEMU emulation BoF at KVM Forum?

I'm very interested too. Thanks for organizing this, Alex! In particular we're interested in the infrastructure work to enable having a single build of QEMU using several different TCG frontends, which is probably preliminary to heterogeneous vCPU emulation. In fact, I was going to send an RFC

  1   2   >