Fix the check for sub-groups availability. It returned false even when device had support for sub-groups

v-klochkov · v-klochkov · commit ac90159911f8 · 2021-03-01T13:32:05.000-08:00
Signed-off-by: Vyacheslav N Klochkov &lt;vyacheslav.n.klochkov@intel.com&gt;
diff --git a/SYCL/SubGroup/helper.hpp b/SYCL/SubGroup/helper.hpp
@@ -154,7 +154,7 @@ void exit_if_not_equal_vec(vec<T, N> val, vec<T, N> ref, const char *name) {
 }
 
 bool core_sg_supported(const device &Device) {
-  return (Device.has_extension("cl_khr_subgroups") ||
-          Device.get_info<info::device::version>().find(" 2.1") !=
-              string_class::npos);
+  if (Device.has_extension("cl_khr_subgroups"))
+    return true;
+  return Device.get_info<info::device::version>() >= "2.1";
 }

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ void exit_if_not_equal_vec(vec<T, N> val, vec<T, N> ref, const char *name) {`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`bool core_sg_supported(const device &Device) {`
`157`		`- return (Device.has_extension("cl_khr_subgroups") \|\|`
`158`		`- Device.get_info<info::device::version>().find(" 2.1") !=`
`159`		`- string_class::npos);`
	`157`	`+ if (Device.has_extension("cl_khr_subgroups"))`
	`158`	`+ return true;`
	`159`	`+ return Device.get_info<info::device::version>() >= "2.1";`
`160`	`160`	`}`