On 11/21/20 9:13 PM, Richard Henderson wrote: > On 11/20/20 1:08 PM, Philippe Mathieu-Daudé wrote: >> Media eXtension Unit is a SIMD extension from Ingenic. >> >> Extract 2900 lines from the huge translate.c to a new file, >> 'vendor-xburst_translate.c.inc'. As there are too many inter- >> dependencies we don't compile it as another object, but >> keep including it in the big translate.o. We gain in code >> maintainability. >> >> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> >> --- >> target/mips/translate.c | 2890 +------------------- >> target/mips/vendor-xburst_translate.c.inc | 2893 +++++++++++++++++++++ >> 2 files changed, 2894 insertions(+), 2889 deletions(-) >> create mode 100644 target/mips/vendor-xburst_translate.c.inc > > Where does the xburst name come from? > It's a little confusing that all of the comments talk about MXU but the > filename is xburst. Perhaps add a comment near the top identifying the > rename, > if you can?
This is in the first comment: * "XBurst® Instruction Set Architecture MIPS eXtension/enhanced Unit * Programming Manual", Ingenic Semiconductor Co, Ltd., revision June 2, 2017 But it appears at the end, and the comment is 340+ lines... I'll move it to the top. Phil. > > > r~ >
