Skip to content

Xcode 5 and 64 bit support#1200

Merged
BradLarson merged 3 commits intoBradLarson:masterfrom
karlvr:xcode5
Oct 9, 2013
Merged

Xcode 5 and 64 bit support#1200
BradLarson merged 3 commits intoBradLarson:masterfrom
karlvr:xcode5

Conversation

@karlvr
Copy link
Copy Markdown
Contributor

@karlvr karlvr commented Sep 18, 2013

I've compiled GPUImage with Xcode 5 and 64 bit support. Presumably this works :-) This is a significant change in here, and that is that the Xcode project minimum deployment target has changed to iOS 6. This may not suit everyone, but I'm guessing that the majority will care about targeting the new devices.

Requires setting the minimum deployment target to 6.0. This is probably not acceptable. I'm not sure what to do here; create multiple targets?
@BradLarson
Copy link
Copy Markdown
Owner

I might hold off on this until I get an actual iPhone 5S in hand. I've done 32->64 bit migration on my Mac applications before, and I know that all kinds of subtle issues can crop up once you run on 64-bit hardware. The GPU in the 5S is also a very different animal, so it will require some closer examination.

BradLarson added a commit that referenced this pull request Oct 9, 2013
Xcode 5 and 64 bit support
@BradLarson BradLarson merged commit 0895487 into BradLarson:master Oct 9, 2013
@BradLarson
Copy link
Copy Markdown
Owner

I'm going to merge this in, but I'll be rolling back the default 64-bit support. Too many people depend on the framework working on targets back to 4.0 to leave that as the default. They can enable this support if needed.

@BradLarson
Copy link
Copy Markdown
Owner

I found one last thing throwing precision errors, and tested everything out under 64-bit. Seems solid, because I think I fixed the few actual problems in there when I migrated this to the Mac earlier (with a 64-bit target there).

I did roll back the architectures in the project and my test application, and have the framework once again deploying to 4.0. People would really start to complain if I removed support for older platforms now, but I may drop 4.0 soon.

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