[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 20 09:14:34 PST 2018


dexonsmith added a comment.

In D55869#1337692 <https://reviews.llvm.org/D55869#1337692>, @rjmccall wrote:

> Okay.  That's also presumably true for open-source runtimes that support ARC; tagging David Chisnall and Jonathan Schleifer to get their input.


`shouldUseARCFunctionsForRetainRelease()` returns `false` for the other runtimes.  This seems like the right thing to do until/unless they add support.

Would make sense for the driver to additionally imply `-fno-objc-convert-messages-to-runtime-calls` for other runtimes as a bigger hammer?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55869/new/

https://reviews.llvm.org/D55869





More information about the cfe-commits mailing list