Integrating sync_wait & sync_wait_with_variant

[hcq9102]

Proposed Changes:

Ref: P2300R5
Integrate sync_wait and sync_wait_with_variant.

Checklist

• I have added a new feature and have added tests to go along with it.

[StellarBot]

Performance test report

HPX Performance

Comparison

BENCHMARK	FORK_JOIN_EXECUTOR	PARALLEL_EXECUTOR	SCHEDULER_EXECUTOR
For Each	(=)	(=)	(=)

Info

Property	Before	After
HPX Datetime	2022-05-31T12:57:29+00:00	2022-09-19T22:13:18+00:00
HPX Commit	d5655f8	9fff673
Clustername	daint	daint
Compiler	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile
Hostname	nid01193	nid00876
Datetime	2022-05-31T15:13:01.357969+02:00	2022-09-20T00:27:10.756628+02:00

Comparison

BENCHMARK	NO-EXECUTOR
Future Overhead - Create Thread Hierarchical - Latch	(=)

Info

Property	Before	After
HPX Datetime	2022-05-31T12:57:29+00:00	2022-09-19T22:13:18+00:00
HPX Commit	d5655f8	9fff673
Clustername	daint	daint
Compiler	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile
Hostname	nid01193	nid00876
Datetime	2022-05-31T15:13:18.026239+02:00	2022-09-20T00:27:27.376396+02:00

Comparison

BENCHMARK	FORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATOR	PARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATOR	SCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATOR
Stream Benchmark - Add	(=)	-	-
Stream Benchmark - Scale	(=)	(=)	(=)
Stream Benchmark - Triad	(=)	(=)	-
Stream Benchmark - Copy	(=)	=	(=)

Info

Property	Before	After
HPX Datetime	2021-11-10T19:14:21+00:00	2022-09-19T22:13:18+00:00
HPX Commit	71d8dbe	9fff673
Clustername	daint	daint
Compiler	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0	/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile
Hostname	nid00120	nid00876
Datetime	2021-11-10T20:28:18.266961+01:00	2022-09-20T00:27:42.743177+02:00

Explanation of Symbols

Symbol	MEANING
=	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…

[hkaiser]

@hcq9102 do we still need to merge #5956 before merging this?

[hcq9102]

@hcq9102 do we still need to merge #5956 before merging this?

I think we don't need #5956 anymore.
But I would like to close #5956 after this PR is merged.

[hkaiser]

@hcq9102 this should be fine now. Please note that I have also rebased (and force-pushed) this PR onto master.

[hkaiser]

LGTM, thanks!

[hkaiser]

bors merge

[bors]

Build succeeded:

• Bors