Skip to content

Commit 3811d7b

Browse files
More documentation/comments
1 parent a888fa8 commit 3811d7b

File tree

2 files changed

+23
-0
lines changed

2 files changed

+23
-0
lines changed

sycl/include/sycl/detail/core.hpp

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,16 @@
88

99
#pragma once
1010

11+
// This is an ongoing experimental activity in its early stage. No code outside
12+
// this project must rely on the behavior of this header file - keep using
13+
// <sycl/sycl.hpp>.
14+
//
15+
// Short-term plan/action items (in no particular order):
16+
// * Update more tests to use this instead of full <sycl/sycl.hpp>.
17+
// * Refactor includes so that transitive dependencies don't bring as much as
18+
// they currently do.
19+
// * Determine what else should be included here.
20+
1121
#include <sycl/accessor.hpp>
1222
#include <sycl/buffer.hpp>
1323
#include <sycl/queue.hpp>

sycl/test-e2e/README.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
* [Creating or modifying tests](#creating-or-modifying-tests)
99
* [LIT feature checks](#lit-feature-checks)
1010
* [llvm-lit parameters](#llvm-lit-parameters)
11+
* [sycl/detail/core.hpp header file](#sycl/detail/core.hpp)
1112

1213
# Overview
1314
This directory contains SYCL-related tests distributed in subdirectories based
@@ -282,3 +283,15 @@ llvm-lit --param dpcpp_compiler=path/to/clang++ --param dump_ir=True \
282283
SYCL/External/RSBench
283284
```
284285

286+
## sycl/detail/core.hpp
287+
288+
While SYCL specification dictates that the only user-visible interface is
289+
`<sycl/sycl.hpp>` header file we found out that as the implementation and
290+
multiple extensions grew, the compile time was getting worse and worse,
291+
negatively affecting our CI turnaround time. We are just starting some efforts
292+
to create a much smaller set of basic feature needed for every SYCL end-to-end
293+
test/program so that this issue could be somewhat mitigated. This activity is in
294+
its early stage and NO production code should rely on it. It WILL be changed as
295+
we go with our experiments. For any code outside of this project only the
296+
`<sycl/sycl.hpp>` must be used until we feel confident to propose an extension
297+
that can provide an alternative.

0 commit comments

Comments
 (0)