Skip to content

Handle the case where LAPACK_*potrf is a macro, not a function (#527)#539

Merged
marcinz merged 1 commit intonv-legate:branch-22.07from
marcinz:cherry-pick-3e193a1
Aug 17, 2022
Merged

Handle the case where LAPACK_*potrf is a macro, not a function (#527)#539
marcinz merged 1 commit intonv-legate:branch-22.07from
marcinz:cherry-pick-3e193a1

Conversation

@marcinz
Copy link
Collaborator

@marcinz marcinz commented Aug 16, 2022

Cherry-pick #527 into branch 22.07.

…gate#527)

This is working around some recent changes to OpenBLAS. Previously we were using
the internal names for functions, e.g. "spotrf_". OpenBLAS changed the
definitions of these internal functions, so in a previous PR we switched to
using the public functions, e.g. "LAPACK_spotrf". These used to be function
symbols, but in the latest update OpenBLAS changed these to be macros.
@marcinz marcinz requested a review from manopapad August 16, 2022 21:17
@marcinz marcinz merged commit ffb37b4 into nv-legate:branch-22.07 Aug 17, 2022
manopapad pushed a commit that referenced this pull request Feb 18, 2025
* update test_clip for np2

* missing comparison

* Apply suggestions from code review

Co-authored-by: Jacob Faibussowitsch <jacob.fai@gmail.com>

---------

Co-authored-by: Jacob Faibussowitsch <jacob.fai@gmail.com>
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