This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v5 3/6] sim: or1k: add or1k target to sim
- From: "Doug Evans via gdb-patches" <gdb-patches at sourceware dot org>
- To: Simon Marchi <simon dot marchi at polymtl dot ca>
- Cc: Stafford Horne <shorne at gmail dot com>, GDB patches <gdb-patches at sourceware dot org>, Openrisc <openrisc at lists dot librecores dot org>, Mike Frysinger <vapier at gentoo dot org>
- Date: Mon, 09 Oct 2017 17:15:08 +0000
- Subject: Re: [PATCH v5 3/6] sim: or1k: add or1k target to sim
- Authentication-results: sourceware.org; auth=none
- Reply-to: Doug Evans <dje at google dot com>
Simon Marchi writes:
> On 2017-10-05 09:49 AM, Stafford Horne wrote:
> > This adds the OpenRISC 32-bit sim target. The OpenRISC sim is a CGEN
> > based sim so the bulk of the code is generated from the .cpu files by
> > CGEN. The engine decode and execute logic in mloop uses scache with
> > pseudo-basic-block extraction and supports both full and fast (switch)
> > modes.
> >
> > The sim does not implement an mmu at the moment. The sim does implement
> > fpu instructions via the common sim-fpu implementation.
> >
> > sim/ChangeLog:
> >
> > 2017-09-13 Stafford Horne <shorne@gmail.com>
> > Peter Gavin <pgavin@gmail.com>
> >
> > * configure.tgt: Add or1k sim.
> > * or1k/Makefile.in: New file.
> > * or1k/configure.ac: New file.
> > * or1k/mloop.in: New file.
> > * or1k/or1k-sim.h: New file.
> > * or1k/or1k.c: New file.
> > * or1k/sim-if.c: New file.
> > * or1k/sim-main.h: New file.
> > * or1k/traps.c: New file.
> > ...
> >
> > +#define CHECK_SPR_FIELD(GROUP, INDEX, FIELD, test) \
> > + do { \
> > + USI field = GET_H_##SYS##_##INDEX##_##FIELD (); \
> > + if (!(test)) { \
> > + sim_io_eprintf(sd, "WARNING: unsupported %s field in %s register: 0x%x\n", \
>
> 80 columns.
fwiw, I don't mind the odd >80 column limit breaker.
Moving the string to the next line may just work,
but if it doesn't it's fine by me to just leave it.