| commit | 94284eb68bb6e461516eb2823cee283eb122ac80 | |
| author | Mark Abraham <mark.j.abraham@gmail.com> | |
| Mon, 27 Jul 2020 12:16:56 +0000 (27 12:16 +0000) | ||
| committer | Mark Abraham <mark.j.abraham@gmail.com> | |
| Mon, 27 Jul 2020 12:16:56 +0000 (27 12:16 +0000) | ||
| tree | c53fe2ebe5f58061808a67fbfb0cc9ae30fbcd4b | treesnapshot (tar.gz zip) |
| parent | 6dd20f9df43d7703f18aed0f1979d02b007cab6c | commitdiff |
Fix RDTSCP handling
Commit 13def2872ae5311d tried to make all builds default to using
RDTSCP, which would have broken non-x86 builds. But it also
deactivated the implementation of RDTSCP support because HAVE_RDTSCP
was left undefined. So all it did was make timing on x86 less
efficient (plus e.g. DLB effects from that).
Used GMX_RDTSCP everywhere. Only the GROMACS project depends on
thread-MPI, so it's reasonable to let a GMX symbol leak in there (and
it's easily fixed if ever needed).
Commit 13def2872ae5311d tried to make all builds default to using
RDTSCP, which would have broken non-x86 builds. But it also
deactivated the implementation of RDTSCP support because HAVE_RDTSCP
was left undefined. So all it did was make timing on x86 less
efficient (plus e.g. DLB effects from that).
Used GMX_RDTSCP everywhere. Only the GROMACS project depends on
thread-MPI, so it's reasonable to let a GMX symbol leak in there (and
it's easily fixed if ever needed).