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

Clean up init_runtime module #634

Merged
merged 1 commit into from
Mar 28, 2023
Merged

Conversation

msimberg
Copy link
Contributor

Part of #16.

Move functionality into detail namespace, move functions into cpp files.

@msimberg msimberg added this to the 0.14.0 milestone Mar 27, 2023
@msimberg msimberg self-assigned this Mar 27, 2023
@msimberg
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Mar 27, 2023
@pika-bot
Copy link
Collaborator

Performance test report

pika Performance

Comparison

BENCHMARKNO-EXECUTOR
Future Overhead - Create Thread Hierarchical - Latch---

Info

PropertyBeforeAfter
pika Datetime2022-09-16T08:18:06+00:002023-03-27T13:54:26+00:00
pika Commit190f1891f12c3b
Clusternamedaintdaint
Envfile
Hostnamenid00074nid00013
Compiler/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1
Datetime2022-09-16T10:25:01.976661+02:002023-03-27T16:00:36.806616+02:00

Explanation of Symbols

SymbolMEANING
=No performance change (confidence interval within ±1%)
(=)Probably no performance change (confidence interval within ±2%)
(+)/(-)Very small performance improvement/degradation (≤1%)
+/-Small performance improvement/degradation (>10%)
++/--Large performance improvement/degradation (>10%)
+++/---Very large performance improvement/degradation (>10%)
?Probably no change, but quite large uncertainty (confidence interval with ±5%)
??Unclear result, very large uncertainty (±10%)
???Something unexpected…

@bors
Copy link
Contributor

bors bot commented Mar 27, 2023

@msimberg msimberg force-pushed the cleanup-init_runtime branch from 98a4f7d to 487a7b0 Compare March 28, 2023 12:19
@msimberg
Copy link
Contributor Author

bors try

@pika-bot
Copy link
Collaborator

Performance test report

pika Performance

Comparison

BENCHMARKNO-EXECUTOR
Future Overhead - Create Thread Hierarchical - Latch---

Info

PropertyBeforeAfter
pika Commit190f18998cfe91
pika Datetime2022-09-16T08:18:06+00:002023-03-28T12:19:14+00:00
Hostnamenid00074nid00598
Envfile
Datetime2022-09-16T10:25:01.976661+02:002023-03-28T14:25:12.513453+02:00
Clusternamedaintdaint
Compiler/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1

Explanation of Symbols

SymbolMEANING
=No performance change (confidence interval within ±1%)
(=)Probably no performance change (confidence interval within ±2%)
(+)/(-)Very small performance improvement/degradation (≤1%)
+/-Small performance improvement/degradation (>10%)
++/--Large performance improvement/degradation (>10%)
+++/---Very large performance improvement/degradation (>10%)
?Probably no change, but quite large uncertainty (confidence interval with ±5%)
??Unclear result, very large uncertainty (±10%)
???Something unexpected…

bors bot added a commit that referenced this pull request Mar 28, 2023
@bors
Copy link
Contributor

bors bot commented Mar 28, 2023

try

Build succeeded:

@msimberg msimberg force-pushed the cleanup-init_runtime branch from 487a7b0 to d411954 Compare March 28, 2023 13:34
@msimberg
Copy link
Contributor Author

bors merge

@pika-bot
Copy link
Collaborator

Performance test report

pika Performance

Comparison

BENCHMARKNO-EXECUTOR
Future Overhead - Create Thread Hierarchical - Latch--

Info

PropertyBeforeAfter
pika Datetime2022-09-16T08:18:06+00:002023-03-28T13:34:42+00:00
pika Commit190f189aeb2afe
Datetime2022-09-16T10:25:01.976661+02:002023-03-28T15:40:24.707440+02:00
Clusternamedaintdaint
Compiler/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1/apps/daint/SSL/pika/spack/lib/spack/env/clang/clang++ 11.0.1
Hostnamenid00074nid00665
Envfile

Explanation of Symbols

SymbolMEANING
=No performance change (confidence interval within ±1%)
(=)Probably no performance change (confidence interval within ±2%)
(+)/(-)Very small performance improvement/degradation (≤1%)
+/-Small performance improvement/degradation (>10%)
++/--Large performance improvement/degradation (>10%)
+++/---Very large performance improvement/degradation (>10%)
?Probably no change, but quite large uncertainty (confidence interval with ±5%)
??Unclear result, very large uncertainty (±10%)
???Something unexpected…

bors bot added a commit that referenced this pull request Mar 28, 2023
628: Miscellaneous cleanup of debug print macros r=msimberg a=msimberg

- Rename macros to be prefixed by `PIKA_DETAIL`
- Remove unnecessary casts in debug print macro calls
- Add note about meaning of detail to README

634: Clean up `init_runtime` module r=msimberg a=msimberg

Part of #16.

Move functionality into `detail` namespace, move functions into cpp files.

638: Fix option description for `PIKA_WITH_THREAD_DEBUG_INFO` r=msimberg a=msimberg

The option is not implicitly enabled in debug builds.

Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
@bors
Copy link
Contributor

bors bot commented Mar 28, 2023

Build failed (retrying...):

bors bot added a commit that referenced this pull request Mar 28, 2023
634: Clean up `init_runtime` module r=msimberg a=msimberg

Part of #16.

Move functionality into `detail` namespace, move functions into cpp files.

638: Fix option description for `PIKA_WITH_THREAD_DEBUG_INFO` r=msimberg a=msimberg

The option is not implicitly enabled in debug builds.

Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
@bors
Copy link
Contributor

bors bot commented Mar 28, 2023

Build failed (retrying...):

@bors
Copy link
Contributor

bors bot commented Mar 28, 2023

Build succeeded:

@bors bors bot merged commit b6692a7 into pika-org:main Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants