Brijesh-Thakkar commented on issue #18849: URL: https://github.com/apache/nuttx/issues/18849#issuecomment-6516312969
Thanks @xiaoxiang781216, good to know about #20996. I'll drop __ultoa_invert from my side, rebase on top of #20996, and send a PR for the remaining kernel call sites (clock_systime_timespec, uptime_read, clock_basetime) with before/after sizes for nucleo-f072rb, stm32vldiscovery and breadxavr. Side finding: div_const in include/nuttx/lib/math32.h gates its reciprocal-multiply path on CONFIG_HAVE_EXPRESSION_STATEMENT, but nothing defines that symbol anywhere in the tree, so the fast path is dead code. Forcing it on made M0 slightly bigger (+116 B, it pulls in __aeabi_lmul instead), so I'm not relying on it. Should it be removed, or wired up? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
