This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH][SIM] Add RXv2 simulation
- From: Yoshinori Sato <ysato at users dot sourceforge dot jp>
- To: Mike Frysinger <vapier at gentoo dot org>, gdb-patches at sourceware dot org
- Date: Wed, 08 Apr 2015 14:54:28 +0900
- Subject: Re: [PATCH][SIM] Add RXv2 simulation
- Authentication-results: sourceware.org; auth=none
- References: <87384c7a0v dot wl-ysato at users dot sourceforge dot jp> <20150408030943 dot GS16816 at vapier>
At Tue, 7 Apr 2015 23:09:43 -0400,
Mike Frysinger wrote:
>
> [1 <text/plain; utf-8 (quoted-printable)>]
> On 07 Apr 2015 15:00, Yoshinori Sato wrote:
> > This patches added new instructions simulation for RX archtecture.
>
> please add some tests. if you look in sim/testsuite/sim/, you should be able to
> copy over something simple like mcore and then write a few basic .s files to
> test the new insns.
>
OK.
> just style comments otherwise -- i'm not familiar with the RX ISA
>
> > + tprintf("sqrt(%g) = %g\n", da, sqrt(da));
>
> all calls need a space before the (
> tprintf ("sqrt(%g) = %g\n", da, sqrt (da));
>
> this comes up a few times in your patch
>
> > + /* Less than 0.49999 */
>
> GNU style comments have a period at the end followed by two spaces:
> /* Less than 0.49999. */
>
> this comes up a few times in your patch
>
> > + whole_bits ++;
>
> no space before the ++ operator
>
> this comes up a few times in your patch
>
> > + while (! (whole_bits & 0x80000000UL))
>
> no space after the !
>
> > + if (tag) { printf (tag); tag = 0; }
>
> i guess not really a new problem, but you're changing this code so you might as
> well fix it, but this should not be one line
Thanks comment.
I'll fix next patches.
> -mike
> [2 Digital signature <application/pgp-signature (7bit)>]
> No public key for 41633B9FE837F581 created at 2015-04-08T12:09:43+0900 using RSA