diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2013-06-24 01:44:55 +0000 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2013-06-24 01:44:55 +0000 |
| commit | 531d5282c0dda68c4618e0353157e61723334711 (patch) | |
| tree | 5c529c4f5349022e80b7a51cf004f92b4516a630 /sim | |
| parent | cdaf73c68afa3c402b222ae93119667caa2b9969 (diff) | |
| download | gdb-531d5282c0dda68c4618e0353157e61723334711.zip gdb-531d5282c0dda68c4618e0353157e61723334711.tar.gz gdb-531d5282c0dda68c4618e0353157e61723334711.tar.bz2 | |
sim: bfin: speed up all insn testcases slightly
The main body of the "all insn" test is executed once per tested insn, and
we test millions of insns here. Any shrinkage we can do in this loop will
speed things up nicely (since it's multiplied per tested insn).
To that end, simplify the end-of-table test into one less insn, and omit
the SSYNC when we build for the sim. When we build to run on the hardware,
this insn matters, but the sim doesn't have write store buffers in the chip
that might get in the way (memory writes are atomic).
Diffstat (limited to 'sim')
| -rw-r--r-- | sim/testsuite/sim/bfin/ChangeLog | 5 | ||||
| -rw-r--r-- | sim/testsuite/sim/bfin/se_allopcodes.h | 5 |
2 files changed, 8 insertions, 2 deletions
diff --git a/sim/testsuite/sim/bfin/ChangeLog b/sim/testsuite/sim/bfin/ChangeLog index ddcce86..11a3072 100644 --- a/sim/testsuite/sim/bfin/ChangeLog +++ b/sim/testsuite/sim/bfin/ChangeLog @@ -1,5 +1,10 @@ 2013-06-23 Mike Frysinger <vapier@gentoo.org> + * se_allopcodes.h (_match): Simplify register test to one less insn. + Omit the SSYNC insn when compiling for the sim. + +2013-06-23 Mike Frysinger <vapier@gentoo.org> + * testutils.inc: Trim trailing whitespace. 2013-06-17 Mike Frysinger <vapier@gentoo.org> diff --git a/sim/testsuite/sim/bfin/se_allopcodes.h b/sim/testsuite/sim/bfin/se_allopcodes.h index 8f96125..796d5c4 100644 --- a/sim/testsuite/sim/bfin/se_allopcodes.h +++ b/sim/testsuite/sim/bfin/se_allopcodes.h @@ -102,8 +102,7 @@ _match: se_all_load_table /* is this the end of the table? */ - R4 = 0; - CC = R4 == R7; + CC = R7 == 0; IF CC jump _new_instruction; /* is the opcode (R0) greater than the 2nd entry in the table (R6) */ @@ -168,8 +167,10 @@ _legal_instruction: _next_instruction: se_all_next_insn +.ifdef BFIN_JTAG /* Make sure the opcode isn't in a write buffer */ SSYNC; +.endif R1 = P5; RETX = R1; |