This is an archive of the discontinued LLVM Phabricator instance.

[ObjC][ARC] Emit the retainRV marker as a module flag instead of named metadata.
ClosedPublic

Authored by ahatanak on Apr 4 2019, 7:08 PM.

Details

Summary

https://reviews.llvm.org/D60302 made changes in the clang front-end so that the retainRV marker is emitted as a module flag. This patch implements the corresponding changes in the ARC contract pass and the auto-upgrader.

Diff Detail

Repository
rL LLVM

Event Timeline

ahatanak created this revision.Apr 4 2019, 7:08 PM
steven_wu accepted this revision.Apr 5 2019, 12:51 PM

LGTM once the clang patch is good to land.

This revision is now accepted and ready to land.Apr 5 2019, 12:51 PM
This revision was automatically updated to reflect the committed changes.