Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vendor demangle.hpp header from Boost #1816

Merged
merged 5 commits into from
Feb 26, 2025

Conversation

msimberg
Copy link
Contributor

@msimberg msimberg commented Feb 14, 2025

This pulls in the demangle.hpp header from Boost (https://github.com/boostorg/core/blob/787b03ea9ca35089cecb80c5f2ca37e10046fbaf/include/boost/core/demangle.hpp) and moves it into the gridtools namespace.

I've done this conversion manually, since it's only one header and it hasn't changed in the last 9 years in Boost. The dependency on boost/config.hpp in the original version is only for BOOST_HAS_PRAGMA_ONCE (replaced to use #pragma once unconditionally), BOOST_GCC (check for __GNUC__ directly) and BOOST_NOEXCEPT (use noexcept unconditionally).

This is part one of two for removing boost as a dependency. The second part to remove Boost.Preprocessor is #1817.

Copy link
Contributor

@havogt havogt left a comment

Choose a reason for hiding this comment

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

Great! I am currently adding back a CSCS CI, then we can merge.

@msimberg
Copy link
Contributor Author

cscs-ci run

1 similar comment
@msimberg
Copy link
Contributor Author

cscs-ci run

@havogt havogt merged commit e22017b into GridTools:master Feb 26, 2025
69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants