[Piro] trailing whitespace
[Piro] pass InArgs by reference
[Piro] Fix indentation
[Piro] added missing break statement
[Piro] properly initialize some uninitialized data members
[Piro] properly initialize some local variables
ML: fix example compile if EpetraExt not enabled

Added EpetraExt to list of required libraries for ml_read_MatrixMarket.cpp.
No longer does a trivial stub build if required libraries aren't
enabled.  This is now checked in CMakeLists.txt.

This hopefully fixes dashboard error.
[Belos] whitespace, indentation
[Belos] Remove workaround warning that caused an error in cray++

Fixes bug #6129 <https://software.sandia.gov/bugzilla/show_bug.cgi?id=6129>.

Build/Test Cases Summary
Enabled Packages: Belos
Enabled all Forward Packages
0) MPI_DEBUG => passed: passed=475,notpassed=0 (29.20 min)
1) SERIAL_RELEASE => passed: passed=441,notpassed=0 (11.95 min)
Other local commits for this build/test group: d18e57d
MueLu: fix a clang warning.
MueLu: remove clang macro from headers

Removing the lines
     #ifdef __clang__
     #pragma clang system_header
     #endif
from various MueLu headers.  This check was originally put in from
SIERRA in commit 8deb080ef to address clang warnings.  However, MueLu
now builds cleanly for me under clang with -Wall.

Build/Test Cases Summary
Enabled Packages: MueLu
Enabled all Forward Packages
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
2) MPI_SS_DEBUG => passed: passed=140,notpassed=0 (16.44 min)
3) SERIAL_SS_DEBUG => passed: passed=105,notpassed=0 (15.42 min)
Other local commits for this build/test group: a019d58
Revert "Ifpack: Disable PreconditionerFactory_LL test"

This reverts commit d07f87abe2a60c68490ee27f6d12cde11a1a54a3.

Reverting this commit as it appears the test is working again.
Tpetra: MV (Kokkos ref.): Replaced some use of lclMV_.

lclMV_ is a KokkosClassic::MultiVector.  It is the primary
representation of the MultiVector's data in the classic version of
Tpetra.  In the Kokkos refactor version of Tpetra, lclMV_ is a
secondary view of the local data, that exists for the sake of gradual
porting.  This commit replaces some use of lclMV_ by working with
(new) Kokkos views directly.  It also adds some comments that discuss
semantic differences of methods like get1dView and get1dCopy, from
that of the classic version of MultiVector.
Tpetra: CrsMatrix: Fixed transferAndFillComplete build error.

There were two build issues with the newly added
transferAndFillComplete method in CrsMatrix:

  1. It's a template method, but was added to the _def.hpp file
     without ETI.
  2. It was not added to the Kokkos refactor version of Tpetra.

This commit moves the method to the _decl.hpp file, and adds it to the
Kokkos refactor version of Tpetra.  However, the method as it stands
can't be added to the _decl.hpp file without forcing us to add many
header includes to the _decl.hpp file.  It would also add a whole lot
of code, which would increase build times, even with ETI on.  Thus,
this commit comments out the body of the method, and replaces it with
an unconditional exception throw.  Thus, this commit will NOT pass the
tests.  I will investigate how to avoid templating the method, so that
we can put its implementation in the _def.hpp file.
Tpetra: Added base class for both Import and Export.

Import and Export share many methods.  Furthermore, someone recently
added a templated method to CrsMatrix, whose template parameter can
only be one of two types: Import or Export.  It would make more sense
to add a base class for both Import and Export, so that the new
CrsMatrix method doesn't need to be templated.  (Templated methods of
a templated class don't work (well) with our ETI system, and thus slow
compilation.)  This commit adds a base class, Transfer.  Transfer
lives in the Tpetra::Details namespace for now, because it is an
implementation detail.  I am not sure if we ever want to expose it to
users.
Tpetra: CrsMatrix: transferAndFillComplete is no longer templated.

In both the "classic" and Kokkos refactor versions of CrsMatrix,
the transferAndFillComplete method is no longer templated.  This
lets us put the implementation in the _def files, thus improving
build times when using ETI.

Build/Test Cases Summary
Enabled Packages: Tpetra, KokkosCore, KokkosCompat, KokkosLinAlg, KokkosContainers, KokkosMpiComm
0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Test case SERIAL_RELEASE was not run! => Does not affect push readiness! (-1.00 min)
2) MPI_DEBUG_KOKKOS_REFACTOR => FAILED: passed=70,notpassed=1 => Not ready to push! (1.96 min)
3) SERIAL_RELEASE_KOKKOS_REFACTOR => passed: passed=52,notpassed=0 (13.79 min)
Other local commits for this build/test group: 261f07f, c645e63, 5e66e00
WARNING: Forced the push!