Skip to content

Commit b6320c1

Browse files
committed
Do not default to NEWLAPACK on Apple Accelerate
Because we now know that Accelerate has a problem with `dpstrf()`, let's not default to the new LAPACK version quite yet.
1 parent 85eb21e commit b6320c1

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

src/autodetection.c

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,18 +65,16 @@ const char * autodetect_symbol_suffix(void * handle, const char * suffix_hint) {
6565
// First, search for LP64-mangling suffixes, so that when we are loading libs from an
6666
// CLI environment, (where suffix hints are not easy) we want to give the most stable
6767
// configuration by default.
68-
#if defined(_OS_DARWIN_) && defined(SYMBOL_TRIMMING)
69-
// Apple Accelerate has an updated LAPACK interface, default to that.
70-
// Note that we are making use of our symbol trimming support here to eliminate
71-
// the F77 trailing underscore by starting the string with `\x1a`.
72-
"\x1a$NEWLAPACK",
73-
#endif
7468
"", "_", "__",
7569

7670
// Next, ILP64-mangling suffixes
7771
#if defined(_OS_DARWIN_) && defined(SYMBOL_TRIMMING)
7872
// Once again, search for Accelerate's non-pure-suffixed names
7973
"\x1a$NEWLAPACK$ILP64",
74+
// Apple Accelerate has an updated LAPACK interface.
75+
// Note that we are making use of our symbol trimming support here to eliminate
76+
// the F77 trailing underscore by starting the string with `\x1a`.
77+
"\x1a$NEWLAPACK",
8078
#endif
8179

8280
// Next, search for ILP64-mangling suffixes

0 commit comments

Comments
 (0)