Skip to content

Tpetra: Use explicit default template arguments in TpetraExt_MMHelpers_decl.hpp

James Willenbringrequested to merge
masterleinad:fix_tpetraext_mmhelpers_decl_cuda-8 into develop

Created by: masterleinad

@trilinos/tpetra

Description

This pull request changes the way the default template arguments for the classes Tpetra::CrsWrapper_CrsMatrix and Tpetra::CrsWrapper_GraphBuilder are specified (in the same way as for Tpetra::CrsMatrixStruct in the same file). In particular, using Tpetra::CrsMatrix<> is avoided.

Motivation and Context

The pull requests aims to fix #4264 (closed). Apparently, nvcc-8 chokes on Tpetra::CrsMatrix<>. Specifying any template argument for that class fixes the issue.

Related Issues

How Has This Been Tested?

i can compile deal.II with a configuration that uses a MueLu wrapper successfully. The setup uses nvcc-8, gcc-5.5 and openmpi-3.1.3. In particular, just compiling TpetraExt_MMHelpers_decl.hpp didn't work before and does now.

Checklist

  • My commit messages mention the appropriate GitHub issue numbers.
  • My code follows the code style of the affected package(s).
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the code contribution guidelines for this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • No new compiler warnings were introduced.
  • These changes break backwards compatibility.

Merge request reports