<div dir="ltr"><div dir="ltr"><div>I had this in the back of my mind for some time, but today I gave it a try to see the effects.<br></div><div dir="ltr"><br></div><div>Many functions declared in panda/common.h are only thin wrappers around other functions. E.g.:<br></div><div dir="ltr"><span style="font-family:monospace,monospace">int panda_physical_memory_rw(hwaddr addr, uint8_t *buf, int len, int is_write);<br>hwaddr panda_virt_to_phys(CPUState *env, target_ulong addr);<br>int panda_virtual_memory_rw(CPUState *env, target_ulong addr, uint8_t *buf, int len, int is_write);<br>int panda_virtual_memory_read(CPUState *env, target_ulong addr, uint8_t *buf, int len);<br>int panda_virtual_memory_write(CPUState *env, target_ulong addr, uint8_t *buf, int len);</span></div><div dir="ltr"><br></div><div>Is there any reason these have not been implemented as inlines?</div><div><br></div><div>After inlining them (a 5&#39; task), my reply replay ran ~5% faster (141.7sec -&gt; 133.7sec). Not a huge improvement, but given the effort required for the change, one can&#39;t complain.<br></div><div><br></div><div>Thanks,</div><div>Manolis<br></div></div></div>