ICRAR / libprofit

90%

master: 90%

LAST BUILD ON BRANCH openmp_20

branch: openmp_20

CHANGE BRANCH

x

Reset

• openmp_20
• 174
• appveyor_2
• cmake-revamp
• correct-flux-capturing
• cpu_extensions
• devel
• fftw_wisdom
• libprofit_173
• master
• memory_improvements
• new_features
• offline-mask
• opencl_version_warnings
• return_finesample
• sonar-qube-fixes
• sonarqube
• static-lib
• tmp
• travis-fix
• travis-macos-fixes
• v1.7.0
• v1.7.1
• v1.7.2
• v1.7.3
• v1.7.4
• v1.8.0
• v1.8.1
• v1.8.2
• v1.9.0
• v1.9.1
• v1.9.2
• v1.9.3
• warning_fixes

Build # 252

Build Type

push

travis-ci

Committed by rtobar

Commit Message

Lowering OpenMP version requirement

The only reason why we required OpenMP 3.0 was because we collapse two
"for" statements in a single "parallel for" pragma. It's easy enough to
imitate that with a single for, so if we find ourselves having only
OpenMP 2.0+ then we provide such single-for version of the code to
implement omp_2d_for.

Other than that, there is the extra requirement in OpenMP 2.0 that the
loop variable has to be signed, which we obey in our single-for version
of the omp_2d_for function template.

I don't know how much lower one can go in terms of OpenMP versions, but
this seems reasonable already, as it includes all major compilers: gcc,
clang, MSVC.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

Run Details

1446 of 1605 relevant lines covered (90.09%)

2945200.17 hits per line