Skip to content

Pull request linked to issue #2834

James Willenbring requested to merge kliegeois:develop into develop

Created by: kliegeois

@trilinos/Stokhos @etphipp

Description

Correction of building warnings of issue #2834 (closed). Adding && and || with one mask and one bool. Adding signbit_v function.

How Has This Been Tested?

MacBook-Pro-de-Kim:TrilinosB kimliegeois$ ctest -R Stokhos_SacadoMPVectorUnitTestMaskTraits_MPI_1
Test project /Users/kimliegeois/dev/TrilinosB
    Start 49: Stokhos_SacadoMPVectorUnitTestMaskTraits_MPI_1
1/1 Test #49: Stokhos_SacadoMPVectorUnitTestMaskTraits_MPI_1 ...   Passed    3.35 sec

100% tests passed, 0 tests failed out of 1

Subproject Time Summary:
Stokhos    =   3.35 sec*proc (1 test)

Total Test time (real) =   3.42 sec

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.

Please let me know if it is not OK @mhoemmen .

Merge request reports