rfcs/experimental/blocked_nd_range_ctad/README.md
oneapi::tbb::blocked_nd_range class was introduced as a representation for recursively divisible N-dimensional range for oneTBB parallel algorithms.
This document proposes extending its API with the deduction guides since C++17 to allow dropping the explicit template arguments specification while
creating the object if they can be determined using the arguments provided:
oneapi::tbb::blocked_range<int> range1(0, 100);
oneapi::tbb::blocked_range<int> range2(0, 200);
oneapi::tbb::blocked_range<int> range3(0, 300);
// Since 3 ranges of type int are provided, the type of nd_range
// can be deduced as oneapi::tbb::blocked_nd_range<int, 3>
oneapi::tbb::blocked_nd_range nd_range(range1, range2, range3);
The oneapi::tbb::blocked_nd_range supports the following set of constructors:
template <typename T, unsigned int N>\
class blocked_nd_range {
public:
using value_type;
using dim_range_type = blocked_range<value_type>;
using size_type = typename dim_range_type::size_type;
blocked_nd_range(const dim_range_type& dim0, /*exactly N arguments of type const dim_range_type&*/); // [1]
blocked_nd_range(const value_type (&dim_size)[N], size_type grainsize = 1); // [2]
blocked_nd_range(blocked_nd_range& r, split); // [3]
blocked_nd_range(blocked_nd_range& r, proportional_split proportion); // [4]
};
The constructor [1] is intended to create an n-dimensional interval by providing N one-dimensional ranges. Each element represents the
dimension of the N-dimensional interval being constructed.
It also allows to construct these one-dimensional intervals in-place from braced-inclosed initializer lists:
// Passing blocked_range objects itself
tbb::blocked_range<int> dim_range(0, 100);
tbb::blocked_nd_range<int, 2> nd_range_1(dim_range, tbb::blocked_range<int>(0, 200));
// Constructing in-place from braced-init-lists
tbb::blocked_nd_range<int, 2> nd_range_2({0, 100}, {0, 200, 5});
// Combined approach
tbb::blocked_nd_range<int, 2> nd_range_3(dim_range, {0, 200, 5});
The constructor [2] is intended to create an interval by providing a C-array each element of which represents a size of the corresponding
dimension of the interval being created. This constructor also allows to pass braced-init-list instead of the array from stack:
// Passing array object itself
int sizes[3] = {100, 200, 300};
// Constructing the 3-dim range [0, 100), [0, 200), [0, 300)
tbb::blocked_nd_range<int, 3> nd_range_1(sizes);
// Using the grainsize - each dim range will have grainsize 5
tbb::blocked_nd_range<int, 3> nd_range_2(sizes, 5);
// Passing the braced-init-list
tbb::blocked_nd_range<int, 3> nd_range_3({100, 200, 300});
In case of passing the template arguments explicitly, using a braced-init-list in both constructors [1] and [2] does not introduce any
ambiguity since if the number of braced-init lists provided is always equal to the number of dimensions of the range for constructor [1] and
the number of elements in the braced-init list equal to the number of dimensions for constructor [2].
Constructors [3] and [4] are intended to split the range into two parts. They are part of Range Named Requirements and used internally in the
implementation of oneTBB parallel algorithms.
This paper proposes to add explicit deduction guides for blocked_nd_range class:
// [g1]
template <typename Value, typename... Values>
blocked_nd_range(blocked_range<Value>, blocked_range<Values>...)
-> blocked_nd_range<Value, 1 + sizeof...(Values)>;
This deduction guide corresponds to the constructor [1] for the case of passing N blocked_range objects itself.
It only participates in overload resolution if all of the types in Values are same as Value.
To cover the case while blocked_ranges are passed as braced-init-lists, it is proposed to add a deduction guide taking a set of C-array objects.
There are currently two options how to define the deduction guide (or a function) taking the braced-init-list
of any type- C-array and std::initializer_list. The issue with std::initializer_list is that it does not allow
tracking the size in compile-time.
// [g2]
template <typename Value, unsigned int... Ns>
blocked_nd_range(const Value (&...)[Ns])
-> blocked_nd_range<Value, sizeof...(Ns)>;
This deduction guide only participates in overload resolution if
sizeof...(Ns) > 1),The first constraint is intended to disambiguate between [1] and [2].
See separate section for more details.
The second constraint is intended to accept only the braced-init-lists that can be used to initialize the blocked_range object.
Currently it supports the constructor with 2 parameters, taking begin and end of the interval, and the constructor with 3 parameters, taking
additional grainsize parameter.
The important limitation of the deduction guide [g2] is that all of the elements in the braced-init-list should be of the same type.
It would be impossible to use this constructor for initializing the blocked_range objects of types that are not convertible to size_type
together with the grainsize:
std::vector<int> vector;
// OK, deduced as blocked_nd_range<iterator, 1>
blocked_nd_range range1({vector.begin(), vector.end()});
// FAIL, all items in braced-init-lists should be objects of the same type
// It is impossible to provide grainsize as iterator since iterator is not convertible to size_type
blocked_nd_range range({vector.begin(), vector.end(), /*grainsize = */5});
For the constructor [2], the following deduction guide is proposed:
// [g3]
template <typename Value, unsigned int N>
blocked_nd_range(const Value (&)[N])
For service constructors [3] and [4], the following guides are proposed:
// [g4]
template <typename Value, unsigned int N>
blocked_nd_range(blocked_nd_range<Value, N>, split)
-> blocked_nd_range<Value, N>;
// [g5]
template <typename Value, unsigned int N>
blocked_nd_range(blocked_nd_range<Value, N>, proportional_split)
-> blocked_nd_range<Value, N>;
From the specification perspective, such a deduction guides can be generated as implicit deduction guides, in the same manner as copy and move constructors.
But the current oneTBB implementation, these deduction guides are not generated implicitly, so the explicit guides are required.
Guides [g4] and [g5] are not proposed to be a part of the spec, only a part of oneTBB implementation.
While using the CTAD with blocked_nd_range, there is an ambiguity between two approaches while using a single braced-init-list of size 2 or 3:
blocked_nd_range range1({10, 20});
blocked_nd_range range2({10, 20, 5});
Since the template arguments for blocked_nd_range are not specified, there can be two possible resolutions:
blocked_nd_range<int, 1> and
constructed using the constructor [1].blocked_nd_range<int, 3>
and constructed using the constructor [2].Since it is unclear which resolution should be chosen, current proposal is not to support such use-case in CTAD and require the user to either explicitly
specialize the template arguments, or to use array or blocked_range type itself to initialize the object.
Another interesting issue that should be resolved, is passing the single C-array object of size 2 or 3 to the constructor:
int array[2] = {100, 200};
tbb::blocked_nd_range range(array);
Since the blocked_range is not constructible from C-array and the braced-init-list is not used, the user expects the range to be deduced as
blocked_nd_range<int, 2> and the constructor [2] to be used.
If we add one more explicit deduction guide to support the code above, the single braced-init-list of size 2 or 3 would also match on this guide.
There are the following options how this issue can be resolved:
tbb::blocked_range while using the braced-init-list construction.[1] with "mixed" argumentsThere is a limitation of the deduction guides proposed if the constructor [1] is used with both arguments of exact tbb::blocked_range type
and the braced-init-lists:
tbb::blocked_range<int> dim_range(0, 100);
tbb::blocked_nd_range nd_range(dim_range, {0, 200}, {0, 300}, dim_range);
These arguments would not match nether on the [g1] not [g2] and it is unclear how to define the deduction guide that covers this case.
Current proposal is to keep this scenario a limitation for using the CTAD and always require using the consistent set of parameters - or
the set of braced-init-lists or the set of tbb::blocked_range objects.
The following conditions need to be met to move the feature from experimental to fully supported:
1 in case of mixing blocked_range objects and braced-init-lists.
See separate section for more details.