xiaoxiang781216 commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1057033297
> > so you want I to define all possible built-in function one by one? > > ``` > > 1. The potential number is huge > > > > 2. Where the official list we can reference > > ``` > > > > > > > > > > > > Anyway, what's benefit with the appraoch? > > it's safer and we can do it step-by-step. > > an alternative is to make this change and later add -fno-builtin-foo when foo is found unsafe. maybe it's good enough. i dunno. I think this approach is better because: 1. The compiler can do the optimization by default 2. The special built-in function get document explicitly 3. We can even adjust the implementation to meet the toolchain expectation -- 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: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
