[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2022-01-17 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-1015036986 Ok, let's merge this until we find the better solution, but the patch need rebase first. -- This is an automated message from the Apache Git Service.

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-06-14 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-859310180 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-04-13 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-818575240 > > c++ constructor is called iby start(libc), we can use -e option to let linker point to main(nuttx) directly. > > are you sure about it? it

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-04-13 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-818541982 > > @yamt see my reply. Another approach is switch to our own entry point then all c++ initialization bypass automatically. > > IIRC, the program

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-04-13 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-818492576 @yamt see my reply. Another approach is switch to our own entry point then all c++ initialization bypass automatically. -- This is an automated message

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-03-15 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-799925357 Not yet, but the keypoint for macOS/Linux is same: how to avoid the host startup code initialize NuttX's c++ constructor. The approach selected on

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2021-01-13 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-759941234 @yamt, these two patches is too hack to support macOS: 1. sim: Postpone constructor calls after main on macOS 2. task_startup: Implement

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #2063: Fix libcxx for sim/macOS

2020-10-23 Thread GitBox
xiaoxiang781216 edited a comment on pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#issuecomment-715204427 @yamt could you reorg the PR into three patch to help review: 1. macOS specific fix 2. libcxx specific fix 3. nuttx fix