Add comment specifying need for work-around in accumulators

ndgrigorian · ndgrigorian · commit 48b0e663a61f · 2025-02-25T19:33:29.000-08:00
diff --git a/dpctl/tensor/libtensor/include/kernels/accumulators.hpp b/dpctl/tensor/libtensor/include/kernels/accumulators.hpp
@@ -86,6 +86,7 @@ template <typename srcTy, typename dstTy> struct CastTransformer
 
 template <typename ScanOpT, typename T> struct needs_workaround
 {
+    // work-around needed due to crash in JITing on CPU
     static constexpr bool value =
         std::is_same_v<ScanOpT, sycl::logical_or<T>> ||
         std::is_same_v<ScanOpT, sycl::logical_and<T>>;

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,7 @@ template <typename srcTy, typename dstTy> struct CastTransformer`
`86`	`86`
`87`	`87`	`template <typename ScanOpT, typename T> struct needs_workaround`
`88`	`88`	`{`
	`89`	`+ // work-around needed due to crash in JITing on CPU`
`89`	`90`	`static constexpr bool value =`
`90`	`91`	`std::is_same_v<ScanOpT, sycl::logical_or<T>> \|\|`
`91`	`92`	`std::is_same_v<ScanOpT, sycl::logical_and<T>>;`