Skip to content

Commit 87f5e53

Browse files
committed
Fix defaults
1 parent d9e97ca commit 87f5e53

6 files changed

Lines changed: 13 additions & 14 deletions

File tree

‎benchmark/bench_math.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ namespace
4040
static_cast<std::int64_t>(state.iterations() * size * 2 * sizeof(input_t)));
4141
}
4242

43-
template <typename Op, typename Alloc, alignment aligned = {}>
43+
template <typename Op, typename Alloc, alignment aligned = alignment{}>
4444
void bench_simd(benchmark::State& state)
4545
{
4646
bench_unary<Op, Alloc>(
@@ -78,7 +78,7 @@ namespace
7878
using unaligned_alloc = typename xsimd::test::unaligned_vector<input_t>::allocator_type;
7979

8080
register_bench<Op>("scalar/aligned", bench_scalar<Op, aligned_alloc>);
81-
register_bench<Op>("simd/aligned", bench_simd<Op, aligned_alloc, { .start_aligned = true }>);
81+
register_bench<Op>("simd/aligned", bench_simd<Op, aligned_alloc, alignment { .start_aligned = true }>);
8282
register_bench<Op>("simd/unaligned", bench_simd<Op, unaligned_alloc>);
8383
}
8484

‎include/xsimd_algorithm/builder.hpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,8 @@ namespace xsimd::builder
216216
/// std::array<batch<U, Arch>, batch_arity<U, T>>, both spanning the same element count.
217217
/// When arity is one, a callback over plain batches is accepted as well.
218218
template <
219-
alignment align = {},
220-
unary_options opts = {},
219+
alignment align = alignment{},
220+
unary_options opts = unary_options{},
221221
typename Arch = xsimd::default_arch,
222222
typename T, typename U, typename Func>
223223
XSIMD_INLINE void map_unary(std::span<T const> in, std::span<U> out, Func&& func)

‎include/xsimd_algorithm/math.hpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
namespace xsimd::algo
1717
{
1818
template <
19-
xsimd::builder::alignment align = {},
19+
xsimd::builder::alignment align = xsimd::builder::alignment{},
2020
typename Arch = xsimd::default_arch,
2121
typename T>
2222
void sqrt(std::span<T const> in, std::span<T> out)
@@ -28,7 +28,7 @@ namespace xsimd::algo
2828
}
2929

3030
template <
31-
xsimd::builder::alignment align = {},
31+
xsimd::builder::alignment align = xsimd::builder::alignment{},
3232
typename Arch = xsimd::default_arch,
3333
typename T>
3434
void abs(std::span<T const> in, std::span<T> out)
@@ -40,7 +40,7 @@ namespace xsimd::algo
4040
}
4141

4242
template <
43-
xsimd::builder::alignment align = {},
43+
xsimd::builder::alignment align = xsimd::builder::alignment{},
4444
typename Arch = xsimd::default_arch,
4545
typename T>
4646
void exp(std::span<T const> in, std::span<T> out)

‎test-utils/include/xsimd_test_utils/math_data.hpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ namespace xsimd::test
4141
}
4242
}
4343

44-
template <xsimd::builder::alignment aligned = {}>
44+
template <xsimd::builder::alignment aligned = xsimd::builder::alignment{}>
4545
static void apply_range_simd(std::span<input_t const> in, std::span<output_t> out)
4646
{
4747
constexpr builder::unary_options opts = { .unroll_factor = 4, .pure = Derived::pure };

‎test-utils/include/xsimd_test_utils/math_ops.hpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ namespace xsimd::test
2828

2929
static constexpr auto name = "sqrt";
3030

31-
template <xsimd::builder::alignment aligned = {}>
31+
template <xsimd::builder::alignment aligned = xsimd::builder::alignment{}>
3232
static void apply(std::span<T const> in, std::span<T> out)
3333
{
3434
xsimd::algo::sqrt<aligned>(in, out);
@@ -53,7 +53,7 @@ namespace xsimd::test
5353

5454
static constexpr auto name = "abs";
5555

56-
template <xsimd::builder::alignment aligned = {}>
56+
template <xsimd::builder::alignment aligned = xsimd::builder::alignment{}>
5757
static void apply(std::span<T const> in, std::span<T> out)
5858
{
5959
xsimd::algo::abs<aligned>(in, out);

‎test/test_math.cpp‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616

1717
namespace
1818
{
19-
template <typename Op, typename Alloc, xsimd::builder::alignment aligned = {}>
19+
template <typename Op, typename Alloc, xsimd::builder::alignment aligned = xsimd::builder::alignment{}>
2020
void check_unary_math()
2121
{
2222
// Not a multiple of the batch size, to exercise the tail.
@@ -50,16 +50,15 @@ TEST_CASE_TEMPLATE(
5050
xsimd::test::abs_op<double>,
5151
xsimd::test::exp_op<float>,
5252
xsimd::test::exp_op<double>,
53-
xsimd::test::widen_op<std::int32_t>
54-
)
53+
xsimd::test::widen_op<std::int32_t>)
5554
{
5655
using input_t = typename Op::input_t;
5756
using aligned_allocator = typename xsimd::test::aligned_vector<input_t>::allocator_type;
5857
using unaligned_allocator = typename xsimd::test::unaligned_vector<input_t>::allocator_type;
5958

6059
SUBCASE("aligned without header")
6160
{
62-
check_unary_math<Op, aligned_allocator, { .start_aligned = true }>();
61+
check_unary_math<Op, aligned_allocator, xsimd::builder::alignment { .start_aligned = true }>();
6362
}
6463

6564
SUBCASE("aligned with header")

0 commit comments

Comments
 (0)