On 6/14/19 10:11 AM, Alex Bennée wrote: > +static inline void set_hostaddr(CPUArchState *env, TCGMemOp mo, void *haddr) > +{ > +#ifdef CONFIG_PLUGIN > + if (mo & MO_HADDR) { > + env_tlb(env)->c.hostaddr = haddr; > + } > +#endif > +} > +
Even if we weren't talking about recomputing this in the helper, would an unconditional store be cheaper? r~