Skip to content

Allow setting OpenMP flags using env var#263

Draft
thilinarmtb wants to merge 1 commit intorusty1s:masterfrom
thilinarmtb:env_var_for_setting_openmp_flag
Draft

Allow setting OpenMP flags using env var#263
thilinarmtb wants to merge 1 commit intorusty1s:masterfrom
thilinarmtb:env_var_for_setting_openmp_flag

Conversation

@thilinarmtb
Copy link
Copy Markdown

The flags to enable OpenMP depends on the compiler and it is not always
-fopenmp. This PR add the env. var OPENMP_FLAG and checks if it is set
before applying the default flag.

@thilinarmtb thilinarmtb marked this pull request as draft January 15, 2026 20:20
@thilinarmtb thilinarmtb force-pushed the env_var_for_setting_openmp_flag branch 2 times, most recently from 6251913 to 5131f28 Compare January 15, 2026 21:21
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 15, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.28%. Comparing base (f2d9919) to head (f2c320a).
⚠️ Report is 16 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #263      +/-   ##
==========================================
- Coverage   97.32%   97.28%   -0.04%     
==========================================
  Files          11       11              
  Lines         224      221       -3     
==========================================
- Hits          218      215       -3     
  Misses          6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@thilinarmtb thilinarmtb force-pushed the env_var_for_setting_openmp_flag branch from 5131f28 to f2c320a Compare January 15, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants