Skip to content

Commit

Permalink
libstdc++: testsuite: Reduce max_size_type.cc exec time [PR113175]
Browse files Browse the repository at this point in the history
The adjustment to max_size_type.cc in r14-205-g83470a5cd4c3d2
inadvertently increased the execution time of this test by over 5x due
to making the two main loops actually run in the signed_p case instead
of being dead code.

To compensate, this patch cuts the relevant loops' range [-1000,1000] by
10x as proposed in the PR.  This shouldn't significantly weaken the test
since the same important edge cases are still checked in the smaller range
and/or elsewhere.  On my machine this reduces the test's execution time by
roughly 10x (and 1.6x relative to before r14-205).

	PR testsuite/113175

libstdc++-v3/ChangeLog:

	* testsuite/std/ranges/iota/max_size_type.cc (test02): Reduce
	'limit' to 100 from 1000 and adjust 'log2_limit' accordingly.
	(test03): Likewise.
  • Loading branch information
Patrick Palka committed Jan 3, 2024
1 parent 45c807b commit a138b99
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions libstdc++-v3/testsuite/std/ranges/iota/max_size_type.cc
Original file line number Diff line number Diff line change
Expand Up @@ -199,8 +199,8 @@ test02()
using max_type = std::conditional_t<signed_p, max_diff_t, max_size_t>;
using shorten_type = std::conditional_t<shorten_p, hw_type, max_type>;
const int hw_type_bit_size = sizeof(hw_type) * __CHAR_BIT__;
const int limit = 1000;
const int log2_limit = 10;
const unsigned limit = 100;
const int log2_limit = 7;
static_assert((1 << log2_limit) >= limit);
const int min = (signed_p ? -limit : 0);
const int max = limit;
Expand Down Expand Up @@ -257,8 +257,8 @@ test03()
using max_type = std::conditional_t<signed_p, max_diff_t, max_size_t>;
using base_type = std::conditional_t<toggle_base_p, hw_type, max_type>;
constexpr int hw_type_bit_size = sizeof(hw_type) * __CHAR_BIT__;
constexpr int limit = 1000;
constexpr int log2_limit = 10;
constexpr unsigned limit = 100;
constexpr int log2_limit = 7;
static_assert((1 << log2_limit) >= limit);
const int min = (signed_p ? -limit : 0);
const int max = limit;
Expand Down

0 comments on commit a138b99

Please sign in to comment.