[SYCL] Do not treat sycl::half kernel argument specially

[Fznamznon]

There was some special handling added when all struct kernel arguments
were decomposed since OpenCL doesn't allow half kernel argument. Once
we've implemented conditional decomposition no special handling needed
for half type.

[smanna12]

@Fznamznon, for my own understanding could you please clarify?
sycl::half is sill supported as kernel argument. This patch removes the special handling for half type due to conditional decomposition implementation.

[Fznamznon]

sycl::half is sill supported as kernel argument. This patch removes the special handling for half type due to conditional decomposition implementation.

Yes, it still supported. We no longer need special treatment for sycl::half type. So, after this patch It will be processed as a struct without fields of special types.

[smanna12]

sycl::half is sill supported as kernel argument. This patch removes the special handling for half type due to conditional decomposition implementation.

Yes, it still supported. We no longer need special treatment for sycl::half type. So, after this patch It will be processed as a struct without fields of special types.

Thanks @Fznamznon for the explanation!

[premanandrao]

Change looks good to me. Do we have any run-time tests that use half-types that would tell us if this change is okay?

[Fznamznon]

Change looks good to me. Do we have any run-time tests that use half-types that would tell us if this change is okay?

I believe we have a lot of them. For example this test - https://github.com/intel/llvm-test-suite/blob/intel/SYCL/USM/fill.cpp captures a struct (test_struct) with half value inside by value.