Skip to content

Conversation

@lucbv
Copy link
Contributor

@lucbv lucbv commented Jul 22, 2025

Adding option to request the library and modifying the BLAS logic to skip the Fortran check and correctly define the mangling macro for Accelerate. This last part might be an undue overload of the macro purpose?

Adding option to request the library and modifying the BLAS logic
to skip the Fortran check and correctly define the mangling macro
for Accelerate. This last part might be an undue overload of the
macro purpose?

Signed-off-by: Luc Berger-Vergiat <lberge@sandia.gov>
@lucbv lucbv requested a review from ndellingwood July 22, 2025 23:27
@lucbv lucbv self-assigned this Jul 22, 2025
@lucbv lucbv marked this pull request as draft July 22, 2025 23:27
lucbv added 2 commits July 23, 2025 10:01
This will allow us to make sure the logic we are adding it indeed
working correctly and tested with the regular CI actions.

Signed-off-by: Luc Berger-Vergiat <lberge@sandia.gov>
Actually including the proper header... renaming a bit the github
action to make it clear which one tests Accelerate. Adding CXX flag
to force using the new version of Accelerate and avoid deprecation
warnings.

Signed-off-by: Luc Berger-Vergiat <lberge@sandia.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant