WIP

serge-sans-paille · serge-sans-paille · commit 1770e0ae4855 · 2025-07-22T23:11:24.000+02:00
diff --git a/include/xsimd/arch/common/xsimd_common_arithmetic.hpp b/include/xsimd/arch/common/xsimd_common_arithmetic.hpp
@@ -203,10 +203,9 @@ namespace xsimd
         {
             if (std::is_signed<T>::value)
             {
-                auto mask = (other >> (8 * sizeof(T) - 1));
                 auto self_pos_branch = min(std::numeric_limits<T>::max() - other, self);
                 auto self_neg_branch = max(std::numeric_limits<T>::min() - other, self);
-                return other + select(batch_bool<T, A>(mask.data), self_neg_branch, self_pos_branch);
+                return other + select(other >= 0, self_pos_branch, self_neg_branch);
             }
             else
             {

Original file line number	Diff line number	Diff line change
`@@ -203,10 +203,9 @@ namespace xsimd`
`203`	`203`	`{`
`204`	`204`	`if (std::is_signed<T>::value)`
`205`	`205`	`{`
`206`		`- auto mask = (other >> (8 * sizeof(T) - 1));`
`207`	`206`	`auto self_pos_branch = min(std::numeric_limits<T>::max() - other, self);`
`208`	`207`	`auto self_neg_branch = max(std::numeric_limits<T>::min() - other, self);`
`209`		`- return other + select(batch_bool<T, A>(mask.data), self_neg_branch, self_pos_branch);`
	`208`	`+ return other + select(other >= 0, self_pos_branch, self_neg_branch);`
`210`	`209`	`}`
`211`	`210`	`else`
`212`	`211`	`{`