yamt commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1038734101
> > > > for simple functions like memset it might make sense. for more complex functions, i'm not sure. > > > > > > > > > could you give some unsafe example? > > > > > > printf? > > > > isn't it safer to enable them selectively as we do for alloca? > > > > > > > > > Could you point how alloca is done? Whether to use builtin function should be a global section: either work, or not work base on the test. > > > > > > https://github.com/apache/incubator-nuttx/blob/master/include/alloca.h > > This is just a declaration, not control whether to enable the built-in. iirc, defining foo to __builtin_foo is the gcc-suggested way to enable them selectively. -- 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
