[SYCL] Remove UB in sampler.cpp (intel#1177)

The SYCL 2020 specification states that the instance of a moved SYCL
object is invalid, so any operations after a move exhibit undefined
behaviour. sampler.cpp moves a sampler before hashing the moved object.
In that case, the hasher will extract a nullptr implementation, which
with intel/llvm#6598 will fail an assertion.
Since this is UB, failing the assertion is valid and the UB case should
be removed.
This commit removes the UB operation and removes the unnecessary
opencl_icd requirement in sampler.cpp.

Signed-off-by: Larsen, Steffen <[email protected]>

Author: steffenlarsen

SYCL/Basic/sampler/sampler.cpp

@@ -1,5 +1,5 @@
-// REQUIRES: opencl, opencl_icd
-// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple -fsycl-dead-args-optimization %s -o %t.out %opencl_lib
+// REQUIRES: opencl
+// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple -fsycl-dead-args-optimization %s -o %t.out
 // RUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
@@ -67,7 +67,6 @@ int main() {
   A = std::move(B);
   assert(Hasher(C) == Hasher(A));
   assert(C == A);
-  assert(Hasher(C) != Hasher(B));
 
   SamplerWrapper WrappedSmplr(sycl::coordinate_normalization_mode::normalized,
                               sycl::addressing_mode::repeat,