Version 8 is the first version to support the __thread
keyword, which is required for building lsan for
64-bit ios.
Details
Details
- Reviewers
kcc glider fjricci - Commits
- rGded235a141ef: [sanitizers] Upgrade ios min version to 8
rG32f0ec51d1e5: Upgrade ios min version to 8
rCRT298860: [sanitizers] Upgrade ios min version to 8
rCRT295412: Upgrade ios min version to 8
rL298860: [sanitizers] Upgrade ios min version to 8
rL295412: Upgrade ios min version to 8
Diff Detail
Diff Detail
- Build Status
Buildable 3839 Build 3839: arc lint + arc unit
Event Timeline
Comment Actions
I am in favor of this change (I wanted to raise the deployment target to 8.0 for some time already). @kcc do you still need sanitizers to support deploying to iOS 7?
Comment Actions
Actually, if you don't want to be blocked here, you could consider using the same trick as D29786.
Comment Actions
Turns out that's a bit tricky, because we need a thread-local AllocatorCache, which is a complex type only defined in the allocator source files.
Comment Actions
Actually, I would still like to push towards making this change. There are other issue with supporting ios/iossim 7.0.