Skip to content

Conversation

adigie
Copy link
Contributor

@adigie adigie commented Feb 18, 2025

  • Add PICOLIBC_LIBCXX_VARIANT config with two options:
    • PICOLIBC_LIBCXX_FULL - standard variant
    • PICOLIBC_LIBCXX_NOEXCEPT - variant compiled with -fno-exceptions
  • Link with libstc++_noexcept if variant is available in toolchain and CPP_EXCEPTIONS is disabled.

* Add PICOLIBC_LIBCXX_VARIANT config with two options:
  * PICOLIBC_LIBCXX_FULL - standard variant
  * PICOLIBC_LIBCXX_NOEXCEPT - variant compiled with `-fno-exceptions`
* Link with `libstc++_noexcept` if variant is available in toolchain and
CPP_EXCEPTIONS is disabled.

Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Copy link
Contributor

@keith-packard keith-packard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good now. You might take a look at my libstdc++ module proposal and see how these two interact. #80762

@adigie
Copy link
Contributor Author

adigie commented Feb 20, 2025

This looks good now. You might take a look at my libstdc++ module proposal and see how these two interact. #80762

They should work fine together. I would just change the depend from GLIBCXX_LIBCPP to GLIBCXX_LIBCPP_USE_TOOLCHAIN

Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Apr 22, 2025
@github-actions github-actions bot closed this May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants