Hello, As Richard explained, the purpose of having separate gen_intermediate_code() and gen_intermediate_code_pc() functions per target is to compile-optimize gen_intermediate_code_internal() for the non-_pc case.
Multiple targets were using static rather than static inline though, fix this. I've split these off from my refactorings so that we can backport them to stable, and I'm rebasing my argument refactoring patches on top. No actual performance changes have been benchmarked, these changes serve more to align our targets as clear examples for new targets such as rl78 and bfin. Regards, Andreas Cc: Richard Henderson <r...@twiddle.net> Cc: Edgar E. Iglesias <edgar.igles...@gmail.com> (cris, mblaze) Cc: Michael Walle <mich...@walle.cc> (lm32) Cc: Peter Crosthwaite <peter.crosthwa...@xilinx.com> (mblaze) Cc: Anthony Green <gr...@moxielogic.com> (moxie) Cc: Max Filippov <jcmvb...@gmail.com> (xtensa) Cc: qemu-sta...@nongnu.org Cc: Mike Frysinger <vap...@gentoo.org> (bfin) Andreas Färber (5): target-cris: gen_intermediate_code_internal() should be inlined target-lm32: gen_intermediate_code_internal() should be inline target-microblaze: gen_intermediate_code_internal() should be inline target-moxie: gen_intermediate_code_internal() should be inline target-xtensa: gen_intermediate_code_internal() should be inline target-cris/translate.c | 2 +- target-lm32/translate.c | 5 +++-- target-microblaze/translate.c | 2 +- target-moxie/translate.c | 2 +- target-xtensa/translate.c | 5 +++-- 5 files changed, 9 insertions(+), 7 deletions(-) -- 1.8.1.4
