On Fri, Oct 05, 2012 at 05:00:04PM -0700, Richard Henderson wrote: > This applies with or without the sparc-compare patch set I > recently sent, and it works with the same set of tests. > > I've not had time to do true benchmarking on this, but it > does reduce the size of the generated code:
Experience proves that there is not a direct relation between size of the generated code and the resulting emulation speed, sometimes smaller code means slower emulation (when the code generation/optimization takes too much time), and sometimes bigger code might means faster emulation (think about replacing some helpers by TCG code). As from the user point of view what is important is the emulation speed, I think benchmarks (even simple ones like measuring the boot time of a guest) are essential for this kind of patch. > BEFORE > Translation buffer state: > gen code size 509344/33431552 > TB count 2196/262144 > TB avg target size 22 max=252 bytes > TB avg host size 231 bytes (expansion ratio: 10.3) > cross page TB count 0 (0%) > direct jump count 1153 (52%) (2 jumps=628 28%) > > AFTER > Translation buffer state: > gen code size 418064/33431552 > TB count 2196/262144 > TB avg target size 22 max=252 bytes > TB avg host size 190 bytes (expansion ratio: 8.4) > cross page TB count 0 (0%) > direct jump count 1153 (52%) (2 jumps=628 28%) > > > r~ > > > Richard Henderson (1): > target-sparc: Use TCG registers for windowed registers > > gdbstub.c | 4 +- > linux-user/main.c | 17 ++-- > linux-user/signal.c | 78 +++++++-------- > linux-user/sparc/target_signal.h | 2 +- > linux-user/sparc64/target_signal.h | 2 +- > monitor.c | 2 +- > target-sparc/cpu.c | 3 +- > target-sparc/cpu.h | 20 ++-- > target-sparc/int32_helper.c | 4 +- > target-sparc/ldst_helper.c | 12 +-- > target-sparc/translate.c | 188 > +++++++++++++++++-------------------- > target-sparc/win_helper.c | 56 ++++++----- > 12 files changed, 190 insertions(+), 198 deletions(-) > > -- > 1.7.11.4 > > > -- Aurelien Jarno GPG: 1024D/F1BCDB73 [email protected] http://www.aurel32.net
