Skip to content

Commit 7a286b3

Browse files
author
Raghuveer Devulapalli
committed
Get rid of an unnecessary branch
1 parent 9a09892 commit 7a286b3

File tree

1 file changed

+3
-16
lines changed

1 file changed

+3
-16
lines changed

lib/x86simdsort-spr.cpp

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,7 @@ namespace avx512 {
77
template <>
88
void qsort(_Float16 *arr, size_t size, bool hasnan, bool descending)
99
{
10-
if (descending) { x86simdsortStatic::qsort(arr, size, hasnan, true); }
11-
else {
12-
x86simdsortStatic::qsort(arr, size, hasnan, false);
13-
}
10+
x86simdsortStatic::qsort(arr, size, hasnan, descending);
1411
}
1512
template <>
1613
void qselect(_Float16 *arr,
@@ -19,12 +16,7 @@ namespace avx512 {
1916
bool hasnan,
2017
bool descending)
2118
{
22-
if (descending) {
23-
x86simdsortStatic::qselect(arr, k, arrsize, hasnan, true);
24-
}
25-
else {
26-
x86simdsortStatic::qselect(arr, k, arrsize, hasnan, false);
27-
}
19+
x86simdsortStatic::qselect(arr, k, arrsize, hasnan, descending);
2820
}
2921
template <>
3022
void partial_qsort(_Float16 *arr,
@@ -33,12 +25,7 @@ namespace avx512 {
3325
bool hasnan,
3426
bool descending)
3527
{
36-
if (descending) {
37-
x86simdsortStatic::partial_qsort(arr, k, arrsize, hasnan, true);
38-
}
39-
else {
40-
x86simdsortStatic::partial_qsort(arr, k, arrsize, hasnan, false);
41-
}
28+
x86simdsortStatic::partial_qsort(arr, k, arrsize, hasnan, descending);
4229
}
4330
} // namespace avx512
4431
} // namespace xss

0 commit comments

Comments
 (0)