On Thu, Sep 27, 2012 at 11:24:34AM -0700, Richard Henderson wrote: > On 09/27/2012 10:15 AM, Aurelien Jarno wrote: > > Add a new function tcg_reg_sync() to synchronize the canonical location > > of a temp with the value in the associated register, but without freeing > > it. Rewrite tcg_reg_free() to first call tcg_reg_sync() and then to free > > the register. > > > > Signed-off-by: Aurelien Jarno <[email protected]> > > Reviewed-by: Richard Henderson <[email protected]> > > > +/* sync register 'reg' by saving it to the corresponding temporary */ > > +static inline void tcg_reg_sync(TCGContext *s, int reg) > > Isn't the compiler good enough at deciding inlining? >
Surprisingly the compiler doesn't inline that many things. For example all the tcg_reg_alloc_* functions are used only once, but are not inlined. -- Aurelien Jarno GPG: 1024D/F1BCDB73 [email protected] http://www.aurel32.net
