LLNL / variorum

Vendor-neutral library for exposing power and performance features across diverse architectures

https://variorum.readthedocs.io

MIT License

67 stars 26 forks source link

Fix flags for OpenMP example #556

Closed masterleinad closed 2 months ago

masterleinad commented 3 months ago

Description

The target is using C not C++. Thus, the C OpenMP flags should be used. Without this change, I'm seeing errors like

[...]
gcc: error: unrecognized command-line option '-fopenmp=libomp'; did you mean '-fopenmp-simd'?
[...]

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature/architecture support (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Build/CI update

How Has This Been Tested?

Compiling and running tests on Frontier.

Test before merge:

[x] Lassen

Checklist:

[ ] I have run ./scripts/check-code-format.sh and confirm my code code follows the style guidelines of variorum
[ ] I have added comments in my code
[ ] My changes generate no new warnings (build with -DENABLE_WARNINGS=ON)
[x] New and existing unit tests pass with my changes