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

Optimize compilation #47984

Merged
merged 4 commits into from
Apr 14, 2023
Merged

Optimize compilation #47984

merged 4 commits into from
Apr 14, 2023

Conversation

novikd
Copy link
Member

@novikd novikd commented Mar 24, 2023

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Reduce the number of dependencies in the header files to speed up the build.

@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-build Pull request with build/testing/packaging improvement label Mar 24, 2023

}

using Int128 = wide::integer<128, signed>;
Copy link
Member

Choose a reason for hiding this comment

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

maybe replace with #include <Core/Types_fwd.h>?

Copy link
Member Author

Choose a reason for hiding this comment

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

Unfortunately, I can't move concept is_decimal declaration there so I had to create base/Decimal_fwd.h.

Copy link
Member

Choose a reason for hiding this comment

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

I meant lines 5-16 in this file repeat the same declarations that exist in Types_fwd.h

@nickitat nickitat self-assigned this Mar 26, 2023
@novikd novikd merged commit 352a240 into master Apr 14, 2023
137 checks passed
@novikd novikd deleted the optimize-compilation branch April 14, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-build Pull request with build/testing/packaging improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants