Tpetra: Progress on #57
Created by: mhoemmen
@trilinos/tpetra @trilinos/xpetra @trilinos/muelu @trilinos/stokhos @trilinos/belos
Description
- Xpetra,Tpetra: Purge defaultArgNode; just create a Node instance
Node creation does nothing now, so there's no need to have functions like defaultArgNode (see e.g., #3033 (closed)). I changed Xpetra and Tpetra so they now just create the Node instance.
- Tpetra: Remove GO=int dependency from Bug 5401 test
Map's Bug 5401 ("Bug" refers to bugs tracked in the older Bugzilla tracking system, not the GitHub issue tracking system) test) unnecessarily required that GlobalOrdinal = int. The test actually only requires a signed GlobalOrdinal type. This commit relaxes the unnecessary requirement.
- Tpetra: Deprecate
KokkosClassic::Details::get(Default)Node
Deprecate KokkosClassic::Details::getNode
and KokkosClassic::Details::getDefaultNode
. Change downstream packages not to call these functions. Clean up many Tpetra tests.
- Xpetra: create(Uniform){Local,Contig}MapWithNode Node arg optional
For the following Xpetra functions, the Node argument is now optional:
- createUniformContigMapWithNode
- createContigMapWithNode
- createLocalMapWithNode
This facilitates #57 (removal of Tpetra's Node template parameter, in favor of Kokkos::Device
).
-
Purge use of getNode from TpetraCore and Belos tests
-
Fix build warnings and clean up many tests
Motivation and Context
Tpetra will deprecate and remove Node in favor of Kokkos::Device. If we don't create or use Node, it's easier to deprecate it.
Related Issues
- Part of #57
How Has This Been Tested?
Locally, with Clang on Mac.
Checklist
-
My commit messages mention the appropriate GitHub issue numbers. -
My code follows the code style of the affected package(s). -
I have read the code contribution guidelines for this project. -
All new and existing tests passed. -
No new compiler warnings were introduced.