Results 11 comments of LU-JOHN

The recommended workaround is to use clamp and then convert. Something like: > input = sycl::vec; > > auto clamped = clamp(input,-128.0,127.0); > auto result = clamped.template convert\();

Compling with clang++ from (May 2, 2024) with the command: > clang++ -fsycl test.cpp Produces the error message: > RequiresExtension: Feature requires the following SPIR-V extension: > Either SPV_EXT_relaxed_printf_string_address_space extension...

> Having it use an extension is fine, but it seems like the result still does not pass validation: > > error: line 165: OpenCL.std printf: expected Format storage class...

I can reproduce a different spirv-val error: > error: line 169: OpFunctionCall Function 's parameter count does not match the argument count. > %call_i = OpFunctionCall %uint %printf %47 %49...

PR to update spirv-val to validate printf correctly made in: https://github.com/KhronosGroup/SPIRV-Tools/pull/5667 to fix incorrect validation message: > error: line 169: OpFunctionCall Function 's parameter count does not match the argument...

llvm-spirv updated to use printf instruction from OpenCL.std in https://github.com/KhronosGroup/SPIRV-LLVM-Translator/pull/2581 and has been merged. spirv-val updated to allow printf calls with non-constant format strings in https://github.com/KhronosGroup/SPIRV-Tools/pull/5677 and is awaiting merge.

The OpenCL runtime team will be investigating the missing symbol issue.

> @LU-JOHN thanks! If [KhronosGroup/SPIRV-Tools#5677](https://github.com/KhronosGroup/SPIRV-Tools/pull/5677) finally fixes this issue, could you please add > > ``` > Fixes https://github.com/intel/llvm/issues/11733 > ``` > > to the description of [KhronosGroup/SPIRV-Tools#5677](https://github.com/KhronosGroup/SPIRV-Tools/pull/5677)? Updated description.

Are we allowed to discard a requirement just because they are contradictory? From https://intel.github.io/llvm-docs/design/OptionalDeviceFeatures.html: > For a kernel that is decorated with the [[sycl::reqd_work_group_size(W)]] or [[sycl::reqd_sub_group_size(S)]] attribute, the exception must...

Updated text describes current behavior of sycl_post_link. The previous text for sycl_post_link may need more revision.