Changes

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor

Avoid calling wrapper functions with singleton in some classes

Deprecate `in_parallel` (#6032) (#6582)

* in_parallel: rebase on upstream/develop, resolve conflicts, clang-formatting

* rebase on upstream/develop, resolve conflicts, clang format

* Apply clang-formatting (clang-8.0.1), mv execute_in_serial function

* OpenMP fixes, clang-8 formatting

* Kokkos_OpenMP_Instance.hpp: add ";" to end of class definition

* Address round 3 of DA comments

* Fix typos in deprecation markings

* Fix CI failures in OpenACC, OpenMP, Threads backends

* rm conditional statement

* rm additional `in_parallel` statements

* Correctly flag namespace brace

* Address most recent DA comments

* rm brace at 3108

* rm else branch; change "==" to "=" in TestExecutionSpace

* Add deprecation marking for all `in_parallel`-requiring tests

* TestHPX_InParallel.cpp: rm unused variable "n"

* TestHPX_InParallel.cpp: put "n" inside of Deprecaton marking

* Add dep. marking to `impl_in_parallel_scope`-type statements

* Add deprecation marking to all impl_not_in_parallel_scope

* Address DLG comments

* rm errant "e += i"

* rm second definition of "execute_in_serial"

* Address DA comments

* Fix up function call operator

* Addressing final DA comments

* Apply deprecation markings uniformly

* Put deprecation markings around braced clauses

* Apply suggestions from review

---------

Co-authored-by: Damien L-G <dalg24@gmail.com>

Remove redundant calls in rangepolicy constructors (#6765)

* Removed redudant calls from RangePolicy constructors

* Fixed to call the right constructor