Skip to content

Commit dca88e7

Browse files
committed
Revisit DAG benchmarks
This overhauls the DAG benchmarks to address some long standing issues: - Instead of measuring the combined time taken to generate a graph and all the graphs that come before it (i.e., TDAG -> CDAG -> IDAG), only measure the generation of the graph being benchmarked. - Don't measure benchmark context creation / destruction. - Fix IDAG benchmarks creating host tasks instead of device tasks (i.e., number of devices and p2p copy support had no effect). - Fix scheduler benchmarks comparing against single-threaded TDAG + CDAG generation, instead of all three graphs. - Remove outdated "throttled submission" scheduler benchmarks.
1 parent 1898534 commit dca88e7

File tree

2 files changed

+197
-158
lines changed

2 files changed

+197
-158
lines changed

include/utils.h

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,14 @@
1919
#define CELERITY_DETAIL_UTILS_CAT_2(a, b) a##b
2020
#define CELERITY_DETAIL_UTILS_CAT(a, b) CELERITY_DETAIL_UTILS_CAT_2(a, b)
2121

22+
#define CELERITY_DETAIL_UTILS_NON_COPYABLE(classname) \
23+
classname(const classname&) = delete; \
24+
classname& operator=(const classname&) = delete
25+
26+
#define CELERITY_DETAIL_UTILS_NON_MOVABLE(classname) \
27+
classname(classname&&) = delete; \
28+
classname& operator=(classname&&) = delete
29+
2230
namespace celerity::detail::utils {
2331

2432
/// Like std::move, but move-constructs the result so it does not reference the argument after returning.

0 commit comments

Comments
 (0)