You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/06/14 07:26:04 UTC

[GitHub] [incubator-nuttx] hartmannathan commented on pull request #3900: Revert "Make: use gcc as LD"

hartmannathan commented on pull request #3900:
URL: https://github.com/apache/incubator-nuttx/pull/3900#issuecomment-859650629


   > > Because:
   > > 
   > > * It's very confusing to have cc as LD.
   > 
   > Yes, it's a little bit confusing, but if you want to enable LTO, you must use gcc to drive the link process as far as I know. If we can't find a method to let ld work with LTO, it's better to keep the change as it.
   > 
   > > * I don't see what "-nostartfiles -nodefaultlibs" in LDFLAGS are
   > >   supposed to do when we use LD directly. It would be simpler to
   > >   remove them from our LDFLAGS.
   
   Agreed that Link Time Optimization would be a good thing to make a reality, if possible. We may run into some additional fallout from the 45672c2 change, like what happened with clang (fixed in #3904). But so far it looks good.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org