AppBlade / AppBladeSDK

The SDK for AppBlade; iOS and Android
https://appblade.com
MIT License
23 stars 14 forks source link

Add -mno-thumb to OTHER_CFLAGS #100

Closed floatplane closed 8 years ago

floatplane commented 9 years ago

The framework project sets GCC_THUMB_SUPPORT = NO, but we are getting link errors in Paper that indicate that THUMB instructions are being used:

ld: b/bl/blx thumb2 branch out of range (17017136 max is +/-16MB): from _protobuf_c_out_of_memory_default (0x02418BCC) to _abort@0x00000000 (0x03453524) in '_protobuf_c_out_of_memory_default' from /Users/jenkins/jenkins/workspace/Paper_develop/iOS/ThirdParty/AppBlade/iOS/Framework/build/Release_Production-iphoneos/libAppBlade.a(protobuf-c.o) for architecture armv7

There is anecdotal evidence that GCC_THUMB_SUPPORT doesn't really do anything: https://twitter.com/luzluz/status/184563684423630848

This change adds the -mno-thumb flag to the compiler options.

PTAL @jamesdaniels @mtjhax /cc @petersibley @mliberman