but degrades the performace a bit as jshFlashRead does not inline spiFlashRead any more
That's odd - actually when I tried it, spiFlashRead was never inlined (before or after changes). And it is still only called once afaik.
However you're talking about pretty minimal clock cycle impacts since it's only called once per read (which'll be 16 bytes normally) - again, if it speeds things up noticeably I'm all for it - but I'm not convinced you will see a noticeable increase in read speeds
Espruino is a JavaScript interpreter for low-power Microcontrollers. This site is both a support community for Espruino and a place to share what you are working on.
That's odd - actually when I tried it,
spiFlashRead
was never inlined (before or after changes). And it is still only called once afaik.However you're talking about pretty minimal clock cycle impacts since it's only called once per read (which'll be 16 bytes normally) - again, if it speeds things up noticeably I'm all for it - but I'm not convinced you will see a noticeable increase in read speeds