Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Revert" Intel deprecation flag changes in spack's lib/spack/env/cc and pull in improved solution from spack mainline #1238

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Aug 12, 2024

Summary

This PR resolves #1230. Detailed information on what this PR does and why it is necessary is given in the issue. The improved solution does away with having to specify the diag-disable flags, since these are now set implicitly (and only during spack environment builds, not in downstream applications). Credits to Peter Scheibel from the spack core team for suggesting it and adding the spack unit tests to my implementation.

Testing

  • CI (with oneapi using icx, icpx, ifort and with intel using icc, icpc, ifort)
  • Manual testing on Dom's development machine (blackpearl; with [email protected] using icx, icpx, ifort)

Applications affected

None

Systems affected

All using recent Intel compilers

Dependencies

Issue(s) addressed

Resolves #1230

Checklist

  • This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.

@climbfuji climbfuji self-assigned this Aug 12, 2024
@climbfuji climbfuji marked this pull request as ready for review August 13, 2024 02:19
@climbfuji climbfuji changed the title Feature/spack compiler wrapper vcheck oneapi deprecate warnings spack stack "Revert" Intel deprecation flag changes in spack's lib/spack/env/cc and pull in improved solution from spack mainline Aug 14, 2024
…eature/spack_compiler_wrapper_vcheck_oneapi_deprecate_warnings_spack_stack
Copy link
Collaborator

@RatkoVasic-NOAA RatkoVasic-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@climbfuji climbfuji enabled auto-merge (squash) August 15, 2024 23:05
@climbfuji climbfuji mentioned this pull request Aug 15, 2024
3 tasks
@climbfuji climbfuji merged commit f4b2f7f into JCSDA:develop Aug 16, 2024
8 checks passed
@climbfuji climbfuji deleted the feature/spack_compiler_wrapper_vcheck_oneapi_deprecate_warnings_spack_stack branch August 16, 2024 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
2 participants