Re: [PATCH v2 3/8] target/sparc: Drop inline markers from translate.c

2023-06-28 Thread Richard Henderson
On 6/27/23 10:58, Philippe Mathieu-Daudé wrote: On 21/6/23 20:06, Richard Henderson wrote: Let the compiler decide about inlining. Should we clean that automatically with scripts and forbid via checkpatch? I'm sure there are some places that benefit from inline within C files. And of course

Re: [PATCH v2 3/8] target/sparc: Drop inline markers from translate.c

2023-06-27 Thread Philippe Mathieu-Daudé
On 21/6/23 20:06, Richard Henderson wrote: Let the compiler decide about inlining. Should we clean that automatically with scripts and forbid via checkpatch? Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Richard Henderson --- target/sparc/translate.c | 237 ++

[PATCH v2 3/8] target/sparc: Drop inline markers from translate.c

2023-06-21 Thread Richard Henderson
Let the compiler decide about inlining. Signed-off-by: Richard Henderson --- target/sparc/translate.c | 237 ++- 1 file changed, 111 insertions(+), 126 deletions(-) diff --git a/target/sparc/translate.c b/target/sparc/translate.c index eec6f9ca67..1312c3e94d