Re: [PATCH] disas/nanomips: Convert nanoMIPS disassembler to C

2022-08-01 Thread Peter Maydell
On Mon, 1 Aug 2022 at 04:18, Vince Del Vecchio wrote: > > On Fri, 29 Jul 2022 at 10:18, Peter Maydell wrote: > > ... > > Is it possible to break this down into smaller pieces so it isn't one > > single enormous 5000 line patch ? > > > > I guess partial conversion is likely to run into

RE: [PATCH] disas/nanomips: Convert nanoMIPS disassembler to C

2022-07-31 Thread Vince Del Vecchio via
On Fri, 29 Jul 2022 at 10:18, Peter Maydell wrote: > ... > Is it possible to break this down into smaller pieces so it isn't one single > enormous 5000 line patch ? > > I guess partial conversion is likely to run into compilation difficulties > mid-series; if so we could do "disable the

Re: [PATCH] disas/nanomips: Convert nanoMIPS disassembler to C

2022-07-29 Thread Peter Maydell
On Fri, 29 Jul 2022 at 15:13, Milica Lazarevic wrote: > > C++ features like class, exception handling and function overloading > have been removed and replaced with equivalent C code. > > Signed-off-by: Milica Lazarevic > --- > Please see the discussion about why converting it here: >