Skip to content

Do not use symbol aliases on macOS.#856

Merged
devinamatthews merged 2 commits intomasterfrom
fix-gemmtr-mac
Feb 19, 2025
Merged

Do not use symbol aliases on macOS.#856
devinamatthews merged 2 commits intomasterfrom
fix-gemmtr-mac

Conversation

@devinamatthews
Copy link
Copy Markdown
Member

@devinamatthews devinamatthews commented Feb 8, 2025

Details:

  • The BLAS/CBLAS function ?gemmtr is currently implemented as a symbol alias of the already-existing ?gemmt. This does not work on macOS/Darwin.
  • Instead, use a minimal wrapper function which calls ?gemmt/?cblas_gemmt, but only on macOS.
  • Also clean up the CBLAS prototypes a bit.

Details:
- The BLAS/CBLAS function `?gemmtr` is currently implemented as a symbol alias of the already-existing `?gemmt`. This does not work on macOS/Darwin.
- Instead, duplicate the entire BLAS function with the new name, and use a minimal wrapper function for CBLAS.
- Also clean up the CBLAS prototypes a bit.
@devinamatthews devinamatthews merged commit 14047f6 into master Feb 19, 2025
@devinamatthews devinamatthews deleted the fix-gemmtr-mac branch February 19, 2025 19:31
devinamatthews added a commit that referenced this pull request Jun 7, 2025
Details:
- The BLAS/CBLAS function `?gemmtr` is currently implemented as a symbol alias of the already-existing `?gemmt`. This does not work on macOS/Darwin.
- Instead, use a minimal wrapper function which calls the appropriate existing BLAS/CBLAS function.
- Also clean up the CBLAS prototypes a bit.

(cherry picked from commit 14047f6)
devinamatthews added a commit that referenced this pull request Jun 7, 2025
Details:
- The BLAS/CBLAS function `?gemmtr` is currently implemented as a symbol alias of the already-existing `?gemmt`. This does not work on macOS/Darwin.
- Instead, use a minimal wrapper function which calls the appropriate existing BLAS/CBLAS function.
- Also clean up the CBLAS prototypes a bit.

(cherry picked from commit 14047f6)
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.

1 participant