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

2021-03-23 Thread Paolo Montesel via
Thanks for the feedback, it helped us improve the implementation quite a bit. > > +| rvalue QMARK rvalue COLON rvalue > > +{ > > +@1.last_column = @5.last_column; > > +bool is_64bit = ($3.bit_width == 64) || ($5.bit_width == 64); > > +int bit_width = (is_64bit) ? 64 : 32; > > +if (

Re: [PATCH 3/4] gitlab-ci: Build Hexagon cross-toolchain

2021-03-04 Thread Paolo Montesel via
On Thu, Mar 4, 2021 at 1:10 PM Alex Bennée wrote: > > > Philippe Mathieu-Daudé writes: > > > Add a job to build the Debian based Hexagon cross-toolchain image. > > > > This image requires a lot of compute time, too much for the common > > shared runners. To avoid having the job to timeout, it has

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

2021-03-01 Thread Paolo Montesel via
> > +/* Copy rules */ > > +#define fLSBOLD(VAL) (fGETBIT(0, VAL)) > > +#define fSATH(VAL) fSATN(16, VAL) > > +#define fSATUH(VAL) fSATUN(16, VAL) > > +#define fVSATH(VAL) fVSATN(16, VAL) > > +#define fVSATUH(VAL) fVSATUN(16, VAL) > > +#define fSATUB(VAL) fSATUN(8, VAL) > > +#define fSATB(VAL) fSATN