Skip to content

Commit a06e1ad

Browse files
author
Raghuveer Devulapalli
authored
Merge pull request #118 from intel/revert-110-avx2_arg
Revert "Support for AVX2 argsort/argselect/key-value sort"
2 parents ee4b033 + e86d1a0 commit a06e1ad

16 files changed

+1454
-1906
lines changed

lib/x86simdsort-avx2.cpp

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
// AVX2 specific routines:
22
#include "avx2-32bit-qsort.hpp"
33
#include "avx2-64bit-qsort.hpp"
4-
#include "avx2-32bit-half.hpp"
5-
#include "xss-common-argsort.h"
64
#include "x86simdsort-internal.h"
75

86
#define DEFINE_ALL_METHODS(type) \
@@ -20,17 +18,6 @@
2018
void partial_qsort(type *arr, size_t k, size_t arrsize, bool hasnan) \
2119
{ \
2220
avx2_partial_qsort(arr, k, arrsize, hasnan); \
23-
}\
24-
template <> \
25-
std::vector<size_t> argsort(type *arr, size_t arrsize, bool hasnan) \
26-
{ \
27-
return avx2_argsort(arr, arrsize, hasnan); \
28-
} \
29-
template <> \
30-
std::vector<size_t> argselect( \
31-
type *arr, size_t k, size_t arrsize, bool hasnan) \
32-
{ \
33-
return avx2_argselect(arr, k, arrsize, hasnan); \
3421
}
3522

3623
namespace xss {

lib/x86simdsort.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -189,12 +189,12 @@ DISPATCH_ALL(partial_qsort,
189189
(ISA_LIST("avx512_skx", "avx2")))
190190
DISPATCH_ALL(argsort,
191191
(ISA_LIST("none")),
192-
(ISA_LIST("avx512_skx", "avx2")),
193-
(ISA_LIST("avx512_skx", "avx2")))
192+
(ISA_LIST("avx512_skx")),
193+
(ISA_LIST("avx512_skx")))
194194
DISPATCH_ALL(argselect,
195195
(ISA_LIST("none")),
196-
(ISA_LIST("avx512_skx", "avx2")),
197-
(ISA_LIST("avx512_skx", "avx2")))
196+
(ISA_LIST("avx512_skx")),
197+
(ISA_LIST("avx512_skx")))
198198

199199
#define DISPATCH_KEYVALUE_SORT_FORTYPE(type) \
200200
DISPATCH_KEYVALUE_SORT(type, uint64_t, (ISA_LIST("avx512_skx")))\

0 commit comments

Comments
 (0)