[PATCH][RX] instructions test set.

[Mike Frysinger]

On 25 Dec 2015 23:19, Yoshinori Sato wrote:
> --- /dev/null
> +++ b/sim/testsuite/sim/rx/abs.s
> @@ -0,0 +1,25 @@
> +# RX testcase for abs
> +# mach:	all

i think you want to use "rx rxv2" instead of "all"

> --- /dev/null
> +++ b/sim/testsuite/sim/rx/allinsn.exp
> @@ -0,0 +1,67 @@
> +# Renesas RX simulator testsuite
> +
> +set all "rx rxv2"
> +set global_ld_options "-Ttext=0x01000000 --defsym __stack=0x01800000"

move these inside of the "if" statement

> +if [istarget rx-*-*] then {

drop the "then" keyword

> +    run_sim_test abs.s  $all

instead of listing them all out, look at how testsuite/sim/msp430/allinsn.exp
is written using a loop and glob.  i think passing down $all for all of the
tests should be fine, even the later ones where you pass rxv2.  the framework
should skip things as needed.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20151225/26ef8a0f/attachment.sig>