Skip to content

Re-enable Clang executor tests#7254

Closed
hkaiser wants to merge 1 commit intomasterfrom
clang_executor_tests
Closed

Re-enable Clang executor tests#7254
hkaiser wants to merge 1 commit intomasterfrom
clang_executor_tests

Conversation

@hkaiser
Copy link
Copy Markdown
Contributor

@hkaiser hkaiser commented May 6, 2026

  • those were accidentally disabled

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 6, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@hkaiser hkaiser force-pushed the clang_executor_tests branch 3 times, most recently from b076bc3 to f5ffd6a Compare May 7, 2026 02:29
@StellarBot
Copy link
Copy Markdown
Collaborator

Performance test report

HPX Performance

Comparison

BENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTOR
For Each(=)(=)---

Info

PropertyBeforeAfter
HPX Datetime2026-03-09T14:08:29+00:002026-05-07T02:29:48+00:00
HPX Commit0eeca864773507
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Clusternamerostamrostam
Datetime2026-03-09T09:15:24.034803-05:002026-05-06T23:21:34.003418-05:00
Envfile
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu

Comparison

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

Info

PropertyBeforeAfter
HPX Datetime2026-03-09T14:08:29+00:002026-05-07T02:29:48+00:00
HPX Commit0eeca864773507
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Clusternamerostamrostam
Datetime2026-03-09T09:17:15.638328-05:002026-05-06T23:23:16.698019-05:00
Envfile
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu

Comparison

BENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATOR
Stream Benchmark - Add=(=)---
Stream Benchmark - Scale(=)-----
Stream Benchmark - Triad(=)-----
Stream Benchmark - Copy(=)+++---

Info

PropertyBeforeAfter
HPX Datetime2026-03-09T18:50:37+00:002026-05-07T02:29:48+00:00
HPX Commitba89f5d4773507
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Clusternamerostamrostam
Datetime2026-03-09T17:49:10.837937-05:002026-05-06T23:24:18.308475-05:00
Envfile
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu

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 (≤5%)
++/--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…

- those were accidentally disabled
- remove use of deprecated execution::execute

Signed-off-by: Hartmut Kaiser <hartmut.kaiser@gmail.com>
@hkaiser hkaiser force-pushed the clang_executor_tests branch from f5ffd6a to 83d91b6 Compare May 7, 2026 15:08
@StellarBot
Copy link
Copy Markdown
Collaborator

Performance test report

HPX Performance

Comparison

BENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTOR
For Each(=)(=)---

Info

PropertyBeforeAfter
HPX Commit0eeca867065176
HPX Datetime2026-03-09T14:08:29+00:002026-05-07T15:08:05+00:00
Clusternamerostamrostam
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu
Envfile
Datetime2026-03-09T09:15:24.034803-05:002026-05-07T10:16:10.605509-05:00

Comparison

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

Info

PropertyBeforeAfter
HPX Commit0eeca867065176
HPX Datetime2026-03-09T14:08:29+00:002026-05-07T15:08:05+00:00
Clusternamerostamrostam
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu
Envfile
Datetime2026-03-09T09:17:15.638328-05:002026-05-07T10:17:55.726028-05:00

Comparison

BENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATOR
Stream Benchmark - Add(=)(=)---
Stream Benchmark - Scale(=)-----
Stream Benchmark - Triad(=)-----
Stream Benchmark - Copy(=)+++---

Info

PropertyBeforeAfter
HPX Commitba89f5d7065176
HPX Datetime2026-03-09T18:50:37+00:002026-05-07T15:08:05+00:00
Clusternamerostamrostam
Compiler/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8/opt/apps/llvm/18.1.8/bin/clang++ 18.1.8
Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.edu
Envfile
Datetime2026-03-09T17:49:10.837937-05:002026-05-07T10:18:58.892506-05: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 (≤5%)
++/--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…

@charan-003 charan-003 mentioned this pull request May 7, 2026
@hkaiser
Copy link
Copy Markdown
Contributor Author

hkaiser commented May 7, 2026

This is superseded by #7257

@hkaiser hkaiser closed this May 7, 2026
@hkaiser hkaiser added this to the 2.0.0 milestone May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants