Revert "Extrema: Only use get_iterator_value for non-numeric types."

This reverts commit 2588854.

Bug 2492864

Author: griwes

thrust/system/detail/generic/extrema.inl

@@ -167,23 +167,13 @@ ForwardIterator min_element(thrust::execution_policy<DerivedPolicy> &exec,
 
   typedef typename thrust::iterator_traits<ForwardIterator>::value_type      InputType;
   typedef typename thrust::iterator_traits<ForwardIterator>::difference_type IndexType;
-  
-  thrust::tuple<InputType, IndexType> initial;
-  if (std::numeric_limits<InputType>::is_specialized)
-  {
-    initial = thrust::tuple<InputType, IndexType>(std::numeric_limits<InputType>::max(), -1);
-  }
-  else
-  {
-    initial = thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec), first), 0);
-  }
-  
+
   thrust::tuple<InputType, IndexType> result =
     thrust::reduce
       (exec,
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))),
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))) + (last - first),
-       initial,
+       thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec), first), 0),
        detail::min_element_reduction<InputType, IndexType, BinaryPredicate>(comp));
 
   return first + thrust::get<1>(result);
@@ -214,23 +204,13 @@ ForwardIterator max_element(thrust::execution_policy<DerivedPolicy> &exec,
 
   typedef typename thrust::iterator_traits<ForwardIterator>::value_type      InputType;
   typedef typename thrust::iterator_traits<ForwardIterator>::difference_type IndexType;
-  
-  thrust::tuple<InputType, IndexType> initial;
-  if (std::numeric_limits<InputType>::is_specialized)
-  {
-    initial = thrust::tuple<InputType, IndexType>(std::numeric_limits<InputType>::lowest(), -1);
-  }
-  else
-  {
-    initial = thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec), first), 0);
-  }
 
   thrust::tuple<InputType, IndexType> result =
     thrust::reduce
       (exec,
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))),
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))) + (last - first),
-       initial,
+       thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec),first), 0),
        detail::max_element_reduction<InputType, IndexType, BinaryPredicate>(comp));
 
   return first + thrust::get<1>(result);
@@ -261,25 +241,15 @@ thrust::pair<ForwardIterator,ForwardIterator> minmax_element(thrust::execution_p
 
   typedef typename thrust::iterator_traits<ForwardIterator>::value_type      InputType;
   typedef typename thrust::iterator_traits<ForwardIterator>::difference_type IndexType;
-  typedef thrust::tuple<InputType, IndexType> AccumulatorType;
-  
-  thrust::tuple<AccumulatorType, AccumulatorType> initial;
-  if (std::numeric_limits<InputType>::is_specialized)
-  {
-    initial = thrust::make_tuple(AccumulatorType(std::numeric_limits<InputType>::max(), -1), AccumulatorType(std::numeric_limits<InputType>::lowest(), -1));
-  }
-  else
-  {
-    initial = detail::duplicate_tuple<InputType, IndexType>()(thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec), first), 0));
-  }
 
   thrust::tuple< thrust::tuple<InputType,IndexType>, thrust::tuple<InputType,IndexType> > result = 
     thrust::transform_reduce
       (exec,
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))),
        thrust::make_zip_iterator(thrust::make_tuple(first, thrust::counting_iterator<IndexType>(0))) + (last - first),
        detail::duplicate_tuple<InputType, IndexType>(),
-       initial,
+       detail::duplicate_tuple<InputType, IndexType>()(
+         thrust::tuple<InputType, IndexType>(thrust::detail::get_iterator_value(derived_cast(exec),first), 0)),
        detail::minmax_element_reduction<InputType, IndexType, BinaryPredicate>(comp));
 
   return thrust::make_pair(first + thrust::get<1>(thrust::get<0>(result)), first + thrust::get<1>(thrust::get<1>(result)));