Skip to content

Commit 26afea8

Browse files
author
Raghuveer Devulapalli
committed
Revert "Build issues on macOS: deduce the pointer to arg with templates"
This reverts commit a3f6753.
1 parent 1558efa commit 26afea8

File tree

1 file changed

+18
-28
lines changed

1 file changed

+18
-28
lines changed

src/xss-common-argsort.h

Lines changed: 18 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -291,15 +291,14 @@ X86_SIMD_SORT_INLINE arrsize_t partition_avx512(type_t *arr,
291291
template <typename vtype,
292292
typename argtype,
293293
int num_unroll,
294-
typename type_t = typename vtype::type_t,
295-
typename argtype_t = typename argtype::type_t>
296-
X86_SIMD_SORT_INLINE arrsize_t argpartition_unrolled(type_t *arr,
297-
argtype_t *arg,
298-
arrsize_t left,
299-
arrsize_t right,
300-
type_t pivot,
301-
type_t *smallest,
302-
type_t *biggest)
294+
typename type_t = typename vtype::type_t>
295+
X86_SIMD_SORT_INLINE arrsize_t partition_avx512_unrolled(type_t *arr,
296+
arrsize_t *arg,
297+
arrsize_t left,
298+
arrsize_t right,
299+
type_t pivot,
300+
type_t *smallest,
301+
type_t *biggest)
303302
{
304303
if (right - left <= 8 * num_unroll * vtype::numlanes) {
305304
return partition_avx512<vtype, argtype>(
@@ -423,12 +422,9 @@ X86_SIMD_SORT_INLINE arrsize_t argpartition_unrolled(type_t *arr,
423422
return l_store;
424423
}
425424

426-
template <typename vtype,
427-
typename argtype,
428-
typename type_t = typename vtype::type_t,
429-
typename argtype_t = typename argtype::type_t>
425+
template <typename vtype, typename type_t>
430426
X86_SIMD_SORT_INLINE type_t get_pivot_64bit(type_t *arr,
431-
argtype_t *arg,
427+
arrsize_t *arg,
432428
const arrsize_t left,
433429
const arrsize_t right)
434430
{
@@ -472,12 +468,9 @@ X86_SIMD_SORT_INLINE type_t get_pivot_64bit(type_t *arr,
472468
}
473469
}
474470

475-
template <typename vtype,
476-
typename argtype,
477-
typename type_t = typename vtype::type_t,
478-
typename argtype_t = typename argtype::type_t>
471+
template <typename vtype, typename argtype, typename type_t>
479472
X86_SIMD_SORT_INLINE void argsort_64bit_(type_t *arr,
480-
argtype_t *arg,
473+
arrsize_t *arg,
481474
arrsize_t left,
482475
arrsize_t right,
483476
arrsize_t max_iters)
@@ -497,10 +490,10 @@ X86_SIMD_SORT_INLINE void argsort_64bit_(type_t *arr,
497490
arr, arg + left, (int32_t)(right + 1 - left));
498491
return;
499492
}
500-
type_t pivot = get_pivot_64bit<vtype, argtype>(arr, arg, left, right);
493+
type_t pivot = get_pivot_64bit<vtype>(arr, arg, left, right);
501494
type_t smallest = vtype::type_max();
502495
type_t biggest = vtype::type_min();
503-
arrsize_t pivot_index = argpartition_unrolled<vtype, argtype, 4>(
496+
arrsize_t pivot_index = partition_avx512_unrolled<vtype, argtype, 4>(
504497
arr, arg, left, right + 1, pivot, &smallest, &biggest);
505498
if (pivot != smallest)
506499
argsort_64bit_<vtype, argtype>(
@@ -510,12 +503,9 @@ X86_SIMD_SORT_INLINE void argsort_64bit_(type_t *arr,
510503
arr, arg, pivot_index, right, max_iters - 1);
511504
}
512505

513-
template <typename vtype,
514-
typename argtype,
515-
typename type_t = typename vtype::type_t,
516-
typename argtype_t = typename argtype::type_t>
506+
template <typename vtype, typename argtype, typename type_t>
517507
X86_SIMD_SORT_INLINE void argselect_64bit_(type_t *arr,
518-
argtype_t *arg,
508+
arrsize_t *arg,
519509
arrsize_t pos,
520510
arrsize_t left,
521511
arrsize_t right,
@@ -536,10 +526,10 @@ X86_SIMD_SORT_INLINE void argselect_64bit_(type_t *arr,
536526
arr, arg + left, (int32_t)(right + 1 - left));
537527
return;
538528
}
539-
type_t pivot = get_pivot_64bit<vtype, argtype>(arr, arg, left, right);
529+
type_t pivot = get_pivot_64bit<vtype>(arr, arg, left, right);
540530
type_t smallest = vtype::type_max();
541531
type_t biggest = vtype::type_min();
542-
arrsize_t pivot_index = argpartition_unrolled<vtype, argtype, 4>(
532+
arrsize_t pivot_index = partition_avx512_unrolled<vtype, argtype, 4>(
543533
arr, arg, left, right + 1, pivot, &smallest, &biggest);
544534
if ((pivot != smallest) && (pos < pivot_index))
545535
argselect_64bit_<vtype, argtype>(

0 commit comments

Comments
 (0)