> On Nov 18, 2015, at 4:21 PM, John McCall <[email protected]> wrote: > > rjmccall added inline comments. > > ================ > Comment at: include/clang/Basic/ObjCRuntime.h:182 > @@ +181,3 @@ > + switch (getKind()) { > + case FragileMacOSX: return false; > + case MacOSX: return getVersion() >= VersionTuple(10, 10); > ---------------- > I went ahead and asked Greg, and he agreed that it's best to not enable this > on the Mac fragile runtime. The functions exist, but they aren't profitable > to call, other than a small code-size decrease. Ah cool. Thanks for checking.
I’m working on updating the patch now with your feedback. Should have a new version soon. I will now also add tests to ensure the fragile runtime does not get this change, since i’d been missing that before. Thanks, Pete > > > http://reviews.llvm.org/D14737 > > > _______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
