Fix compilation warnings and clang suggestions

(From Sierra)

Build/Test Cases Summary
Enabled Packages: SEACASAprepro_lib, SEACASIoss
Disabled Packages: Panzer,PyTrilinos,SEACASExo2mat,SEACASMat2exo,Optika,TrilinosCouplings
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_DEBUG_SS => passed: passed=11,notpassed=0 (19.27 min)
3) SERIAL_RELEASE_SS => passed: passed=12,notpassed=0 (20.31 min)
Implementation of Belos Block GCRODR now supports both real and complex
scalar types. To achieve this, Mark Hoemmen added some LAPACK calls in
Teuchos (specifically GEEV and GGEVX); these calls will always return
the real and imaginary parts of the eigenvalues separately, regardless
of the scalar type. Also, UNGQR is called instead of ORGQR.

Build/Test Cases Summary
Enabled Packages: Belos, TeuchosNumerics
Enabled all Forward Packages
0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => passed: passed=614,notpassed=0 (31.72 min)
MueLu: fixing implicit PR rebalance interface inconsistencies
MueLu: added few comments to sched.template

So that you can understand the if clauses even after substitutions.
"achievedTol" function for Block GCRODR; returns achieved tolerance
from the last solve.
Merge branch 'master' of software.sandia.gov:/space/git/Trilinos
Fixed badly placed parenthesis in TEST_FAIL_AND_THROW macro for serial builds.
Problem detected by clang.