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

Fix Clang sapphire rapids march flag #4435

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

imciner2
Copy link
Contributor

A copy-paste error slipped in during the changes to how the Sapphire Rapids architecture flags were guarded in #4192, so it turns out Clang was never being told to use the Sapphire Rapids, and instead was just using Cooperlake instead.

@martin-frbg martin-frbg added this to the 0.3.27 milestone Jan 16, 2024
@martin-frbg
Copy link
Collaborator

Thanks, totally missed that - seems sometimes the mind reads what it expects to read rather than what the actual code/text is

@martin-frbg martin-frbg merged commit 452741b into OpenMathLib:develop Jan 16, 2024
62 of 64 checks passed
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