Kenneth Moreland authored Feb 11, 2020 and Kitware Robot committed Feb 11, 2020

7e49c113

 Mark the find_package(benchmark) with required

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1942

Kenneth Moreland authored Feb 11, 2020 and Kitware Robot committed Feb 11, 2020

f7709a8d

 Mark deprecated superclasses of WorkletMapTopology with VTKM_DEPRECATED

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1955

Robert Maynard authored Feb 11, 2020 and Kitware Robot committed Feb 11, 2020

1f61c500 Remove non-atomic ops from BitField unit test.
5565848d

 Use a dynamic strategy for openmp 1D scheduling.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1925

Robert Maynard authored Jan 29, 2020 and Kitware Robot committed Jan 29, 2020

36161b42 Renamed to SchedulingRange to follow VTK-m naming convention
40c0c0fe

 Allow for easier overloads of scheduling_range

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1952

This is needed to support custom types that need 3D scheduling
or to call custom methods to get the 1D scheduling size

Kenneth Moreland authored Jan 22, 2020 and Kitware Robot committed Jan 22, 2020

fb01d38a

 Disable deprecated attribute when using nvcc under VS

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1949

Robert Maynard authored Jan 22, 2020 and Kitware Robot committed Jan 22, 2020

dc86ac20

 Avoid a GCC 6.1 compiler regression that occurs when openmp is enabled

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1950

The nvcc compiler under visual studio seems to give the error `attribute
does not apply to any entity` when you try to use the `[[deprecated]]`
attribute. So disable for this compiler configuration.

Robert Maynard authored Jan 21, 2020 and Kitware Robot committed Jan 21, 2020

3e99b177

 Correct host/device warnings in CellSetPermutation found by VTK

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1947

Robert Maynard authored Jan 20, 2020 and Kitware Robot committed Jan 20, 2020

3f02558f

 correct warnings in color table

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1946

Robert Maynard authored Jan 15, 2020 and Kitware Robot committed Jan 15, 2020

b9516c11 Correct CellSetStructured compile failures
00235874 Suppress more warning types from thirdparty includes
a52af2d1 Correct double to float warning in CellAspectFrobeniusMetric
cf5ebfb1 Suppress warning about extension use, since all compilers support it
27739660 Add missing constructors/assignment operators
123f8b01 Mark virtual destructors as override where applicable
54118dfc

 Use noexcept instead of throw() as it was deprecated in c++11

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Kenneth Moreland <kmorel@sandia.gov>
Merge-request: !1943

Having a custom assignment operator means that the compiler
isn't required to generate the implicit copy constructor.
This makes sure they are constructed.

Kenneth Moreland authored Jan 09, 2020 and Kitware Robot committed Jan 09, 2020

aa59224f

 More testing of long lists

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1941

Add the REQUIRED flag to the find_package command to find the Google
benchmark package. Without that flag, the find_package command will
happily pass if the benchmarks are not found.

However, if benchmarks is not found, then the CMake configure will then
fail because the benchmark::benchmark library is not defined. That is a
confusing error to get. Since benchmark is not a typical thing installed
on a system, this can be problematic.

We encountered some issues with `ListIndexOf` for longer lists with
repeated elements. Add some tests for that.

Kenneth Moreland authored Jan 09, 2020 and Kitware Robot committed Jan 09, 2020

b534bdb1

 Remove std::conditional from List.h

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1940

The `std::conditional` utility is very convenient, but it unfortunately
means that the compiler has to evaluate both the true and false types
even though one is guaranteed to be thrown out. This is problematic
because it requires the compiler to do a lot more work then necessary.
It is especially dumb when introducing the conditionals to reduce the
number of cases being evaluated, as was much of the case in List.h.

Allison Vacanti authored Jan 08, 2020 and Kitware Robot committed Jan 08, 2020

46b7155b Add 64-bit CUDA atomic store.
539f6e5a Port benchmarking framework to Google Benchmark.
39d981bc

 Shorten names returned by testing's TypeName utility.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1936

Eg:

```
vtkm::Float32                                       -> F32
vtkm::Int64                                         -> I64
vtkm::Vec< vtkm::Float32, 3 >                       -> Vec3f_32
vtkm::Vec< vtkm::Pair< vtkm::Int32, vtkm::Float64 > -> Vec<Pair<I32, F64>>
```

This makes the benchmark names a lot shorter to keep rows tabular
results on
a single line.

Kenneth Moreland authored Jan 07, 2020 and Kitware Robot committed Jan 07, 2020

60c57303 Fix name clash of template parameter and field
4a52a3f7 Shorter StorageTag for ArrayHandleZip
4a626b2e Shorter storage tag for ArrayHandleView
4c8c28d1 Shorter storage tag for ArrayHandleReverse
bfd21dfa Declare StorageTags with VTKM_ALWAYS_EXPORT
1ec716e5 Shorten storage tag for ArrayHandlePermutation
ca0ad1dc Shorten storage tag for ArrayHandleIndex
7bd5802d

 Shorter storage tag for ArrayHandleGroupVecVariable
...

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robert.maynard@kitware.com>
Merge-request: !1937

Also discovered that many C++ compilers have trouble giving warnings
for partial specialization of classes marked as deprecated. Fix
the problem by instead deprecating the items in the class.

This modifier is required on OSX for dynamic_cast to work properly
across libraries.