[SYCL] test new complex support by group_algorithms

[cperkinsintel]

These test updates support intel/llvm#5394 where we expand some of the group algorithms to support std::complex data type.

Signed-off-by: Chris Perkins [email protected]

[cperkinsintel]

/verify with intel/llvm#5394

[steffenlarsen]

Tests look good, but I fear it could cause failures on devices without aspect::fp64. Do we test on any such devices?

[cperkinsintel]

@steffenlarsen - I added protection to check for aspect::fp64 before testing std::complex

[steffenlarsen]

@steffenlarsen - I added protection to check for aspect::fp64 before testing std::complex

You have! But the problem is that the kernel using fp64 operations may be bundled together with the other kernels. This means when the test kernels are built they may include this kernel and fail.

[cperkinsintel]

/verify with intel/llvm#5394

[vladimirlaz]

@steffenlarsen - I added protection to check for aspect::fp64 before testing std::complex

You have! But the problem is that the kernel using fp64 operations may be bundled together with the other kernels. This means when the test kernels are built they may include this kernel and fail.

The potential fixes for the situation may be:

• move fp64 code to separate test;
• use -fsycl-device-code-split==per_kernel option for building test app.

[cperkinsintel]

I believe this is ready for merge.

[vladimirlaz]

I believe this is ready for merge.

don't we want to add -fsycl-device-code-split=per_kernel to avoid potential failures on devices that do not support fp64?

[steffenlarsen]

LGTM! Thanks for adding the split. 😄