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

Split explicit instantiation of 'ArrayData' and 'Array2Data' in several files #1873

Merged

Conversation

grospelliergilles
Copy link
Member

The goal is to speed-up the compilation of these files.
With these modification, it take 3x7s instead of 20s.

…NUMERIC_DATATYPE' en trois parties pour instantier en 3 fois.

Cela permet de répartir la compilation sur plusieurs fichiers et de
l'accélérer.
Supprime aussi les fichiers 'ArrayData.h' et 'Array2Data.h' plus utilisés.
@grospelliergilles grospelliergilles added arcane Arcane Component refactoring Code refactoring and cleanup labels Dec 26, 2024
@grospelliergilles grospelliergilles self-assigned this Dec 26, 2024
@grospelliergilles grospelliergilles force-pushed the dev/gg-separate-some-template-explicit-instantiation branch from 997f5e6 to c4f4c58 Compare December 26, 2024 09:17
@grospelliergilles grospelliergilles force-pushed the dev/gg-separate-some-template-explicit-instantiation branch from c4f4c58 to 34e8d08 Compare December 26, 2024 09:29
Copy link

codecov bot commented Dec 26, 2024

Codecov Report

Attention: Patch coverage is 63.17568% with 218 lines in your changes missing coverage. Please review.

Project coverage is 70.26%. Comparing base (fc129a4) to head (34e8d08).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
arcane/src/arcane/impl/ArrayData.inst.h 55.35% 90 Missing and 10 partials ⚠️
arcane/src/arcane/impl/Array2Data.inst.h 67.28% 61 Missing and 10 partials ⚠️
arcane/src/arcane/impl/internal/Array2Data.h 64.36% 25 Missing and 6 partials ⚠️
arcane/src/arcane/impl/internal/ArrayData.h 75.00% 14 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1873      +/-   ##
==========================================
- Coverage   70.30%   70.26%   -0.04%     
==========================================
  Files        2272     2276       +4     
  Lines      163777   163773       -4     
  Branches    18863    18857       -6     
==========================================
- Hits       115142   115074      -68     
- Misses      41809    41869      +60     
- Partials     6826     6830       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@grospelliergilles grospelliergilles merged commit c9b1098 into main Dec 26, 2024
31 checks passed
@grospelliergilles grospelliergilles deleted the dev/gg-separate-some-template-explicit-instantiation branch December 31, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arcane Arcane Component refactoring Code refactoring and cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant