Incorporating code review comments.

Signed-off-by: Gail Lyons <[email protected]>

Author: glyons-intel

sycl/CMakeLists.txt

@@ -14,7 +14,7 @@ include(AddSYCLExecutable)
 set(SYCL_MAJOR_VERSION 2)
 set(SYCL_MINOR_VERSION 1)
 set(SYCL_PATCH_VERSION 0)
-set(SYCL_DEV_ABI_VERSION 5)
+set(SYCL_DEV_ABI_VERSION 4)
 if (SYCL_ADD_DEV_VERSION_POSTFIX)
   set(SYCL_VERSION_POSTFIX "-${SYCL_DEV_ABI_VERSION}")
 endif()

sycl/source/detail/device_impl.cpp

@@ -224,40 +224,28 @@ bool device_impl::has(aspect Aspect) const {
   switch (Aspect) {
   case aspect::host:
     return is_host();
-    break;
   case aspect::cpu:
     return is_cpu();
-    break;
   case aspect::gpu:
     return is_gpu();
-    break;
   case aspect::accelerator:
     return is_accelerator();
-    break;
   case aspect::fp16:
     return has_extension("cl_khr_fp16");
-    break;
   case aspect::fp64:
     return has_extension("cl_khr_fp64");
-    break;
   case aspect::int64_base_atomics:
     return has_extension("cl_khr_int64_base_atomics");
-    break;
   case aspect::int64_extended_atomics:
     return has_extension("cl_khr_int64_extended_atomics");
-    break;
   case aspect::image:
     return get_info<info::device::image_support>();
-    break;
   case aspect::online_compiler:
     return get_info<info::device::is_compiler_available>();
-    break;
   case aspect::online_linker:
     return get_info<info::device::is_linker_available>();
-    break;
   case aspect::queue_profiling:
     return get_info<info::device::queue_profiling>();
-    break;
   default:
     throw runtime_error("This device aspect has not been implemented yet.",
                         PI_INVALID_DEVICE);