Skip to content

Commit 9659324

Browse files
committed
Clarify test
Signed-off-by: Julian Oppermann <julian.oppermann@codeplay.com>
1 parent f2de935 commit 9659324

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

sycl/test-e2e/KernelCompiler/kernel_compiler_sycl_jit.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -203,15 +203,14 @@ int test_build_and_run() {
203203
syclex::properties{syclex::build_options{flags}, syclex::save_log{&log},
204204
syclex::registered_kernel_names{"ff_templated<int>"}});
205205

206-
// extern "C" was used, so the name "ff_cp" is not mangled and can be used
207-
// directly.
206+
// extern "C" was used, so the name "ff_cp" is implicitly known.
208207
sycl::kernel k = kbExe2.ext_oneapi_get_kernel("ff_cp");
209208

210-
// The templated function name will have been mangled.
209+
// The templated function name was registered.
211210
sycl::kernel k2 = kbExe2.ext_oneapi_get_kernel("ff_templated<int>");
212211

213-
// We can also use the mangled name. This escape hatch might be removed in the
214-
// future.
212+
// We can also use the compiler-generated names directly.
213+
assert(kbExe2.ext_oneapi_has_kernel("__sycl_kernel_ff_cp"));
215214
assert(
216215
kbExe2.ext_oneapi_has_kernel("_Z26__sycl_kernel_ff_templatedIiEvPT_S1_"));
217216

0 commit comments

Comments
 (0)