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 utest compilation #4442

Merged
merged 1 commit into from
Jan 18, 2024
Merged

Fix utest compilation #4442

merged 1 commit into from
Jan 18, 2024

Conversation

pbo-linaro
Copy link
Contributor

Introduced recently when adding new test cases for ZSCAL

  • include cblas is needed for cblas_zscal
  • ASSERT macro does not exist
  • missing closing )

Introduced recently when adding new test cases for ZSCAL

- include cblas is needed for cblas_zscal
- ASSERT macro does not exist
- missing closing )
@pbo-linaro
Copy link
Contributor Author

pbo-linaro commented Jan 18, 2024

This was found thanks to our windows-arm64 CI monitoring OpenBLAS status for this platform.
https://gitlab.com/Linaro/windowsonarm/nightly/-/jobs/5953927509

@pbo-linaro
Copy link
Contributor Author

Regression added in e48627c.
@martin-frbg If you have a chance to merge this, that would be appreciated. Thanks a lot :)

@martin-frbg
Copy link
Collaborator

Thanks - had realized this but not gotten around to fixing it (normally it should have triggered CI as well, but the relevant Azure jobs are silently failing on some kind of VS installation problem right now)

@martin-frbg martin-frbg added this to the 0.3.27 milestone Jan 18, 2024
@martin-frbg martin-frbg merged commit 500442c into OpenMathLib:develop Jan 18, 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