From 1dd444ea71d30cc3a1eab7af0ba3f6a0357ae93c Mon Sep 17 00:00:00 2001 From: Benoit Steiner Date: Wed, 18 Nov 2015 11:37:58 -0800 Subject: Avoid using the version of TensorIntDiv optimized for 32-bit integers when the divisor can be equal to one since it isn't supported. --- unsupported/test/cxx11_tensor_intdiv.cpp | 27 ++++++++++++++++++++++++--- 1 file changed, 24 insertions(+), 3 deletions(-) (limited to 'unsupported/test/cxx11_tensor_intdiv.cpp') diff --git a/unsupported/test/cxx11_tensor_intdiv.cpp b/unsupported/test/cxx11_tensor_intdiv.cpp index 343b37dbd..fd6d27ae1 100644 --- a/unsupported/test/cxx11_tensor_intdiv.cpp +++ b/unsupported/test/cxx11_tensor_intdiv.cpp @@ -14,8 +14,29 @@ void test_signed_32bit() { + // Divide by one + const Eigen::internal::TensorIntDivisor div(1); + + for (int32_t j = 0; j < 25000; ++j) { + const int32_t fast_div = j / div; + const int32_t slow_div = j / 1; + VERIFY_IS_EQUAL(fast_div, slow_div); + } + + // Standard divide by 2 or more + for (int32_t i = 2; i < 25000; ++i) { + const Eigen::internal::TensorIntDivisor div(i); + + for (int32_t j = 0; j < 25000; ++j) { + const int32_t fast_div = j / div; + const int32_t slow_div = j / i; + VERIFY_IS_EQUAL(fast_div, slow_div); + } + } + + // Optimized divide by 2 or more for (int32_t i = 2; i < 25000; ++i) { - const Eigen::internal::TensorIntDivisor div(i); + const Eigen::internal::TensorIntDivisor div(i); for (int32_t j = 0; j < 25000; ++j) { const int32_t fast_div = j / div; @@ -42,7 +63,7 @@ void test_unsigned_32bit() void test_signed_64bit() { - for (int64_t i = 2; i < 25000; ++i) { + for (int64_t i = 1; i < 25000; ++i) { const Eigen::internal::TensorIntDivisor div(i); for (int64_t j = 0; j < 25000; ++j) { @@ -56,7 +77,7 @@ void test_signed_64bit() void test_unsigned_64bit() { - for (uint64_t i = 2; i < 25000; ++i) { + for (uint64_t i = 1; i < 25000; ++i) { const Eigen::internal::TensorIntDivisor div(i); for (uint64_t j = 0; j < 25000; ++j) { -- cgit v1.2.3