reviewer feedback

Signed-off-by: Chris Perkins <chris.perkins@intel.com>

Author: cperkinsintel

sycl/include/CL/sycl/group_algorithm.hpp

@@ -10,7 +10,6 @@
 #include <CL/__spirv/spirv_ops.hpp>
 #include <CL/__spirv/spirv_types.hpp>
 #include <CL/__spirv/spirv_vars.hpp>
-#include <CL/sycl/ONEAPI/atomic.hpp>
 #include <CL/sycl/ONEAPI/functional.hpp>
 #include <CL/sycl/detail/spirv.hpp>
 #include <CL/sycl/detail/type_traits.hpp>

sycl/include/CL/sycl/known_identity.hpp

@@ -9,6 +9,9 @@
 #pragma once
 
 #include <CL/sycl/detail/type_traits.hpp>
+#include <functional>
+#include <limits>
+#include <type_traits>
 
 __SYCL_INLINE_NAMESPACE(cl) {
 namespace sycl {

sycl/test/on-device/group_algorithms_sycl2020/all_of.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/any_of.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/exclusive_scan.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/group_broadcast.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/inclusive_scan.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/none_of.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/permute_select.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/reduce.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -I . -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out

sycl/test/on-device/group_algorithms_sycl2020/shift_left_right.cpp

@@ -1,6 +1,4 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -o %t.out
-// TODO: re-enable HOST execution line when this test is moved to
-// llvm-test-suite XUN: %HOST_RUN_PLACEHOLDER %t.out
 // RUN: %CPU_RUN_PLACEHOLDER %t.out
 // RUN: %GPU_RUN_PLACEHOLDER %t.out
 // RUN: %ACC_RUN_PLACEHOLDER %t.out