Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport performance fix for C++17 #1618

Merged
merged 6 commits into from
Mar 25, 2021

Commits on Mar 23, 2021

  1. Daint: Update daint modules after upgrade (CUDA 11.0) (GridTools#1613)

    2 tests disabled:
    - boundary_conditions_gpu because of GridTools#1522
    - test_tuple.cu conversion constructor for Clang 11.0.0 because of GridTools#1615
    havogt committed Mar 23, 2021
    Configuration menu
    Copy the full SHA
    7faf8fc View commit details
    Browse the repository at this point in the history
  2. add dom.sh

    havogt committed Mar 23, 2021
    Configuration menu
    Copy the full SHA
    d4205ae View commit details
    Browse the repository at this point in the history
  3. reenable some tests

    havogt committed Mar 23, 2021
    Configuration menu
    Copy the full SHA
    e1bdf74 View commit details
    Browse the repository at this point in the history
  4. fix performance (GridTools#1611)

    With C++17, `as_const` was added into the standard and if `as_const` is not called qualified, we end up with `std::as_const` (which is also a better match) in most cases because there is some `std` inside one of the template arguments.
    
    By qualifying `as_const`, performance is fixed.
    
    For GTBench, performance is identical with `-std=c++14` and `-std=c++17` now.
    lukasm91 authored and havogt committed Mar 23, 2021
    Configuration menu
    Copy the full SHA
    8870975 View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2021

  1. Configuration menu
    Copy the full SHA
    bb7b07d View commit details
    Browse the repository at this point in the history

Commits on Mar 25, 2021

  1. ups...

    havogt authored Mar 25, 2021
    Configuration menu
    Copy the full SHA
    e60ab57 View commit details
    Browse the repository at this point in the history