Add comment to needs_workaround in accumulators.hpp

ndgrigorian · ndgrigorian · commit fa525a704261 · 2025-02-27T18:47:22.000-08:00
Documents the ticket reported for the JIT failure on CPU
diff --git a/dpctl/tensor/libtensor/include/kernels/accumulators.hpp b/dpctl/tensor/libtensor/include/kernels/accumulators.hpp
@@ -88,7 +88,8 @@ 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
+    // workaround needed due to crash in JITing on CPU
+    // remove when CMPLRLLVM-65813 is resolved
     static constexpr bool value = su_ns::IsSyclLogicalAnd<T, ScanOpT>::value ||
                                   su_ns::IsSyclLogicalOr<T, ScanOpT>::value;
 };

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,8 @@ template <typename srcTy, typename dstTy> struct CastTransformer`
`88`	`88`
`89`	`89`	`template <typename ScanOpT, typename T> struct needs_workaround`
`90`	`90`	`{`
`91`		`- // work-around needed due to crash in JITing on CPU`
	`91`	`+ // workaround needed due to crash in JITing on CPU`
	`92`	`+ // remove when CMPLRLLVM-65813 is resolved`
`92`	`93`	`static constexpr bool value = su_ns::IsSyclLogicalAnd<T, ScanOpT>::value \|\|`
`93`	`94`	`su_ns::IsSyclLogicalOr<T, ScanOpT>::value;`
`94`	`95`	`};`