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

Neoverse code cleanup for building with `-DENABLE_WARNINGS=ON`. #529

Open tpatki opened 6 months ago

tpatki commented 6 months ago

Description

Cleanup for -Werror-unused-parameter , need to figure out how far to propogate the change and check where to test with @amarathe84.

Fixes #526.

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?

[ ] Juno r2
[ ] neoverse system?

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)
[ ] New and existing unit tests pass with my changes

Thank you for taking the time to contribute to Variorum!