Skip to content

fix -lto_library filtering for apple-clang + gfortran on sdk26#5474

Merged
martin-frbg merged 4 commits intoOpenMathLib:developfrom
Chrismarsh:sdk26
Sep 30, 2025
Merged

fix -lto_library filtering for apple-clang + gfortran on sdk26#5474
martin-frbg merged 4 commits intoOpenMathLib:developfrom
Chrismarsh:sdk26

Conversation

@Chrismarsh
Copy link
Copy Markdown
Contributor

This fixes #5473 where the filtering of -lto_library needs to be applied for xcode >= 16

@martin-frbg martin-frbg added this to the 0.3.31 milestone Sep 30, 2025
@martin-frbg
Copy link
Copy Markdown
Collaborator

Uh, sorry for mishandling the initial merge conflict here...

@Chrismarsh
Copy link
Copy Markdown
Contributor Author

Chrismarsh commented Sep 30, 2025

I think I made the pr off of tag 0.3.30 too 😬

All correct now?

@martin-frbg
Copy link
Copy Markdown
Collaborator

I had removed the if gfortran conditional in #5436 already, maybe that's what made github flag a merge conflict where there was none (and unfortunately I compared with a local checkout from before that September 2 commit by mistake).
I'll do a squash-and-merge and hope it will end up looking as originally intended...

@martin-frbg martin-frbg merged commit c1f607c into OpenMathLib:develop Sep 30, 2025
83 of 88 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.

Link error with apple-clang and gfortran: ld: -lto_library library filename must be 'libLTO.dylib'

2 participants