Hi, This series, while boring, helps maintainability.
I simply exploded 60% of the huge target/mips/translate.c, reducing it from 32K lines of code to 13500. The small overhead in the diffstat is due to entries added in MAINTAINERS and license boilerplate addition: 20225 insertions(+), 19987 deletions(-) While being a massive diff, it is a no-brain review using 'git-diff --color-moved=dimmed-zebra' which highlights very few changes: #include and license lines. The exploded new layout, which allows more useful filtering with the get_maintainer.pl script, is: - MIPS ISA, ASE and modules: . isa-micromips_helper.h.inc . isa-nanomips_translate.c.inc . ase-mips16e_translate.c.inc . mod-mips-dsp_helper.c . mod-mips-dsp_helper.h.inc . mod-mips-dsp_translate.c.inc . mod-mips-msa_helper.h.inc . mod-mips-msa_translate.c.inc . mod-mips-msa_helper.c . mod-mips-mt_helper.h.inc - MIPS Vendor Specific: . vendor-loong-simd_helper.c . vendor-loong-lext_translate.c.inc . vendor-loong-simd_helper.h.inc . vendor-loong-simd_translate.c.inc . vendor-tx-mmi_translate.c.inc . vendor-tx_translate.c.inc . vendor-vr54xx_helper.c . vendor-vr54xx_helper.h.inc . vendor-vr54xx_translate.c.inc . vendor-xburst_translate.c.inc There should be no logical code change (only code movement). The series is available at: https://gitlab.com/philmd/qemu.git branches/mips_translate_explode Regards, Phil. Philippe Mathieu-Daudé (26): target/mips: Extract FPU helpers to 'fpu_helper.h' target/mips: Extract MSA helpers to mod-mips-msa_helper.c target/mips: Extract MSA helper definitions target/mips: Extract MSA translation routines target/mips: Rename dsp_helper.c as mod-mips-dsp_helper.c target/mips: Extract DSP helper definitions target/mips: Extract DSP translation routines target/mips: Extract Multi-Threading helper definitions target/mips: Extract Code Compaction ASE translation routines target/mips: Extract the microMIPS ISA helper definitions target/mips: Extract the microMIPS ISA translation routines target/mips: Extract nanoMIPS ISA translation routines target/mips: Extract NEC Vr54xx helpers to vendor-vr54xx_helper.c target/mips: Extract NEC Vr54xx helper definitions target/mips: Extract NEC Vr54xx translation routines target/mips: Rename lmmi_helper.c as loong-simd_helper.c target/mips: Extract Loongson SIMD helper definitions target/mips: Extract Loongson SIMD translation routines target/mips: Extract Loongson EXTensions translation routines target/mips: Extract XBurst Media eXtension Unit translation routines target/mips: Make pipeline 1 multiply opcodes generic target/mips: Extract Toshiba TXx9 translation routines target/mips: Extract Toshiba TX79 multimedia translation routines MAINTAINERS: Add entry for MIPS Loongson TCG MAINTAINERS: Add entry for MIPS Ingenic Xburst TCG MAINTAINERS: Add entry for MIPS Toshiba TCG target/mips/fpu_helper.h | 50 + target/mips/helper.h | 881 +- target/mips/internal.h | 42 - linux-user/mips/cpu_loop.c | 1 + target/mips/fpu_helper.c | 1 + target/mips/gdbstub.c | 1 + target/mips/kvm.c | 1 + target/mips/machine.c | 1 + .../{dsp_helper.c => mod-mips-dsp_helper.c} | 2 + .../{msa_helper.c => mod-mips-msa_helper.c} | 393 + target/mips/op_helper.c | 511 - target/mips/translate.c | 18598 +--------------- ...mi_helper.c => vendor-loong-simd_helper.c} | 0 target/mips/vendor-vr54xx_helper.c | 131 + MAINTAINERS | 20 + target/mips/ase-mips16e_translate.c.inc | 1170 + target/mips/isa-micromips_helper.h.inc | 17 + target/mips/isa-micromips_translate.c.inc | 3316 +++ target/mips/isa-nanomips_translate.c.inc | 4839 ++++ target/mips/meson.build | 8 +- target/mips/mod-mips-dsp_helper.h.inc | 344 + target/mips/mod-mips-dsp_translate.c.inc | 2158 ++ target/mips/mod-mips-msa_helper.h.inc | 443 + target/mips/mod-mips-msa_translate.c.inc | 2218 ++ target/mips/mod-mips-mt_helper.h.inc | 36 + target/mips/vendor-loong-lext_translate.c.inc | 450 + target/mips/vendor-loong-simd_helper.h.inc | 69 + target/mips/vendor-loong-simd_translate.c.inc | 611 + target/mips/vendor-tx-mmi_translate.c.inc | 573 + target/mips/vendor-tx_translate.c.inc | 317 + target/mips/vendor-vr54xx_helper.h.inc | 24 + target/mips/vendor-vr54xx_translate.c.inc | 93 + target/mips/vendor-xburst_translate.c.inc | 2893 +++ 33 files changed, 20225 insertions(+), 19987 deletions(-) create mode 100644 target/mips/fpu_helper.h rename target/mips/{dsp_helper.c => mod-mips-dsp_helper.c} (99%) rename target/mips/{msa_helper.c => mod-mips-msa_helper.c} (94%) rename target/mips/{lmmi_helper.c => vendor-loong-simd_helper.c} (100%) create mode 100644 target/mips/vendor-vr54xx_helper.c create mode 100644 target/mips/ase-mips16e_translate.c.inc create mode 100644 target/mips/isa-micromips_helper.h.inc create mode 100644 target/mips/isa-micromips_translate.c.inc create mode 100644 target/mips/isa-nanomips_translate.c.inc create mode 100644 target/mips/mod-mips-dsp_helper.h.inc create mode 100644 target/mips/mod-mips-dsp_translate.c.inc create mode 100644 target/mips/mod-mips-msa_helper.h.inc create mode 100644 target/mips/mod-mips-msa_translate.c.inc create mode 100644 target/mips/mod-mips-mt_helper.h.inc create mode 100644 target/mips/vendor-loong-lext_translate.c.inc create mode 100644 target/mips/vendor-loong-simd_helper.h.inc create mode 100644 target/mips/vendor-loong-simd_translate.c.inc create mode 100644 target/mips/vendor-tx-mmi_translate.c.inc create mode 100644 target/mips/vendor-tx_translate.c.inc create mode 100644 target/mips/vendor-vr54xx_helper.h.inc create mode 100644 target/mips/vendor-vr54xx_translate.c.inc create mode 100644 target/mips/vendor-xburst_translate.c.inc -- 2.26.2