[AutoUpgrader] Make ArcRuntime Autoupgrader more conservative
authorSteven Wu <stevenwu@apple.com>
Tue, 13 Aug 2019 17:52:21 +0000 (17:52 +0000)
committerSteven Wu <stevenwu@apple.com>
Tue, 13 Aug 2019 17:52:21 +0000 (17:52 +0000)
commit9e51fb6c576205c4ada59675d89443df10b4abff
treef6eef4df6ac416c71078805854f9d25bc7f3d65c
parentbbccb94d02ef70c06583f4804cc52a4678ddbc04
[AutoUpgrader] Make ArcRuntime Autoupgrader more conservative

Summary:
This is a tweak to r368311 and r368646 which auto upgrades the calls to
objc runtime functions to objc runtime intrinsics, in order to make sure
that the auto upgrader does not trigger with up-to-date bitcode.

It is possible for bitcode that is up-to-date to contain direct calls to
objc runtime function and those are not inserted by compiler as part of
ARC and they should not be upgraded. Now auto upgrader only triggers as
when the old style of ARC marker is used so it is guaranteed that it
won't trigger on update-to-date bitcode.

This also means it won't do this upgrade for bitcode from llvm-8 and
llvm-9, which preserves the behavior of those releases. Ideally they
should be upgraded as well but it is more important to make sure
AutoUpgrader will not trigger on up-to-date bitcode.

Reviewers: ahatanak, rjmccall, dexonsmith, pete

Reviewed By: dexonsmith

Subscribers: hiraditya, jkorous, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D66153

llvm-svn: 368730
llvm/include/llvm/IR/AutoUpgrade.h
llvm/lib/Bitcode/Reader/BitcodeReader.cpp
llvm/lib/IR/AutoUpgrade.cpp
llvm/test/Bitcode/upgrade-arc-runtime-calls-new.bc [new file with mode: 0644]
llvm/test/Bitcode/upgrade-arc-runtime-calls.ll