[PATCH v5 3/6] sim: or1k: add or1k target to sim

Doug Evans via gdb-patches gdb-patches@sourceware.org
Mon Oct 9 17:15:00 GMT 2017


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.



More information about the Gdb-patches mailing list