Better icx flags and (indirectly) force -qopenmp w/ icx >= 24.x #17
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tested with ic(p)x 2022.1.0, 2024.2.1 and 2025.0.0.
The two most recent, 2024.2.1 and 2025.0.0, now compile libMesh cleanly.
The oldest, 2022.1.0, produces a warning regarding fast fp math which is well documented over the web, e.g. assimp/assimp#4450 or https://stackoverflow.com/questions/73534181/fast-floating-point-model-broken-on-next-generation-intel-compiler. I chose neither to silence the warning (so the user knows the compiler is broken), nor to switch to precise math (so the user doesn't lose performance) - the (now informed) user can then choose to silence the warning themselves, switch to a different compiler, or simply live with the consequences...
Cheers,
-N