max and min now use MinMaxAtomicSupportFactory

ndgrigorian · ndgrigorian · commit f35d63e2665b · 2023-10-27T11:46:06.000-07:00
These functions were using ArithmeticAtomicSupportFactory, which disables atomics for floating point types
diff --git a/dpctl/tensor/libtensor/source/reductions/reduction_atomic_support.hpp b/dpctl/tensor/libtensor/source/reductions/reduction_atomic_support.hpp
@@ -117,12 +117,12 @@ template <typename fnT, typename T> struct MinMaxAtomicSupportFactory
 };
 
 template <typename fnT, typename T>
-struct MaxAtomicSupportFactory : public ArithmeticAtomicSupportFactory<fnT, T>
+struct MaxAtomicSupportFactory : public MinMaxAtomicSupportFactory<fnT, T>
 {
 };
 
 template <typename fnT, typename T>
-struct MinAtomicSupportFactory : public ArithmeticAtomicSupportFactory<fnT, T>
+struct MinAtomicSupportFactory : public MinMaxAtomicSupportFactory<fnT, T>
 {
 };
 

Original file line number	Diff line number	Diff line change
`@@ -117,12 +117,12 @@ template <typename fnT, typename T> struct MinMaxAtomicSupportFactory`
`117`	`117`	`};`
`118`	`118`
`119`	`119`	`template <typename fnT, typename T>`
`120`		`-struct MaxAtomicSupportFactory : public ArithmeticAtomicSupportFactory<fnT, T>`
	`120`	`+struct MaxAtomicSupportFactory : public MinMaxAtomicSupportFactory<fnT, T>`
`121`	`121`	`{`
`122`	`122`	`};`
`123`	`123`
`124`	`124`	`template <typename fnT, typename T>`
`125`		`-struct MinAtomicSupportFactory : public ArithmeticAtomicSupportFactory<fnT, T>`
	`125`	`+struct MinAtomicSupportFactory : public MinMaxAtomicSupportFactory<fnT, T>`
`126`	`126`	`{`
`127`	`127`	`};`
`128`	`128`