Re: [PATCH][SIM] Add RXv2 simulation

[Mike Frysinger <vapier at gentoo dot org>]

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.

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
-mike

Attachment: signature.asc
Description: Digital signature