Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move away from obj->isa, and wrap the workAroundMacOSXABIBreakingBug... #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jrmiddle
Copy link

...check with a diagnostic ignore (bitmasking tagged pointers is deprecated).

There's likely more work to do around the ABIBreakingBug check; primarily wanted to get this up and running on iOS/Xcode5.0.1. Tested on iOS7.

…check with a diagnostic ignore (bitmasking tagged pointers is deprecated).
if(JK_EXPECT_F(workAroundMacOSXABIBreakingBug)) { goto slowClassLookup; }
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wdeprecated-objc-pointer-introspection"
BOOL workAroundMacOSXABIBreakingBug = (JK_EXPECT_F(((NSUInteger)object) & 0x1)) ? YES : NO;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this wont work on 64bit iOS devices.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

odyth is right. So thank you!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

@jcbertin
Copy link

I have fixed this problem for latest OSes (iOS 8 and MacOS 10.10). See #158.

@GrayWang
Copy link

good,i fix this problem for iphone6(arm64),Thanks!

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.

5 participants