xiaoxiang781216 edited a comment on pull request #5476:
URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1038720267


   > > > for simple functions like memset it might make sense. for more complex 
functions, i'm not sure.
   > > 
   > > 
   > > could you give some unsafe example?
   > 
   > printf?
   
   But it should work as expect from my test. Anyway, we can test to see what 
happen.
   
   > 
   > > > 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.
   


-- 
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


Reply via email to