On 10/05/16 19:21, Richard Henderson wrote: > On 05/10/2016 05:46 AM, Sergey Fedorov wrote: >> From: Sergey Fedorov <serge.f...@gmail.com> >> >> Simplify cpu_exec() by extracting exception handling code out of >> cpu_exec() into a new static inline function cpu_handle_exception(). >> Also make cpu_handle_debug_exception() inline as it is used only once. > > If it's used only once, the compiler is going to do this anyway, and > therefore there's no point in making the change. Let's just leave off > all the inline markers and trust the compiler, eh?
I agree the compiler is smart enough to decide and inline such functions by itself. But actually, I hope such "static inline" in .c file could indicate for a reader of the code that this function is going to be used this way. > > Otherwise, > > Reviewed-by: Richard Henderson <r...@twiddle.net> Kind regards, Sergey