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

2021-04-05 Thread Taylor Simpson
> -Original Message- > From: Alessandro Di Federico > Sent: Thursday, March 18, 2021 12:27 PM > To: Taylor Simpson > Cc: Richard Henderson ; qemu- > de...@nongnu.org; Brian Cain ; bab...@rev.ng; > ni...@rev.ng; phi...@redhat.com > Subject: Re: [PATCH v2 02/10

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

2021-03-18 Thread Alessandro Di Federico via
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: /*

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

2021-03-10 Thread Taylor Simpson
ect: Re: [PATCH v2 02/10] target/hexagon: import README for idef- > parser > > On 2/25/21 7:18 AM, Alessandro Di Federico wrote: > > +Now let's have a quick look at the generated code, line by line. > > + > > +:: > > + > > + tcg_gen_movi_i32(RdV, 0); > >

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

2021-03-01 Thread Alessandro Di Federico via
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 v2 02/10] target/hexagon: import README for idef-parser

2021-02-25 Thread Richard Henderson
On 2/25/21 7:18 AM, Alessandro Di Federico wrote: > +Now let's have a quick look at the generated code, line by line. > + > +:: > + > + tcg_gen_movi_i32(RdV, 0); > + > +This code starts by initializing ``RdV``, since reading from that register > +without initialization will cause a segmentation

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

2021-02-25 Thread Alessandro Di Federico via
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