Apply version constraints to iOS objects by alexreinking · Pull Request #8546 · halide/Halide · GitHub
Skip to content

Apply version constraints to iOS objects#8546

Merged
alexreinking merged 1 commit into
mainfrom
ios-linker
Jan 12, 2025
Merged

Apply version constraints to iOS objects#8546
alexreinking merged 1 commit into
mainfrom
ios-linker

Conversation

@alexreinking

Copy link
Copy Markdown
Member

Fixes linker errors of the form

ld: warning: no platform load command found in 'library.a[2](object.o)', assuming: iOS

If 17 is too restrictive, we can always drop it to be more permissive later.

@alexreinking alexreinking added backport me This change should be backported to release versions release_notes For changes that may warrant a note in README for official releases. labels Jan 7, 2025
@jiawen

jiawen commented Jan 7, 2025

Copy link
Copy Markdown
Contributor

@alexreinking alexreinking merged commit 5f56f68 into main Jan 12, 2025
@vksnk

vksnk commented Apr 17, 2025

Copy link
Copy Markdown
Member

This change seems to cause a linker error in one of our internal projects, the error message it gives is: ld: building for 'iOS-simulator', but linking in object file <object_file_name> built for 'iOS' . I tried to figure out how to fix it myself, but wasn't very successful (I know very little about iOS development, so not really surprising), so would appreciate any help.

@jiawen

jiawen commented Apr 17, 2025

Copy link
Copy Markdown
Contributor

This change seems to cause a linker error in one of our internal projects, the error message it gives is: ld: building for 'iOS-simulator', but linking in object file <object_file_name> built for 'iOS' . I tried to figure out how to fix it myself, but wasn't very successful (I know very little about iOS development, so not really surprising), so would appreciate any help.

Is this an error or just a warning?

This is annoying though. LLVM's Triple treats iOS as an "OS". But ld wants to be more fine grained. Maybe Environment = Simulator might work?

@vksnk

vksnk commented Apr 17, 2025

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport me This change should be backported to release versions release_notes For changes that may warrant a note in README for official releases.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants