[PATCH] D66047: Do not call replaceAllUsesWith to upgrade calls to ARC runtime functions to intrinsic calls
Steven Wu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 12 12:28:18 PDT 2019
steven_wu accepted this revision.
steven_wu added inline comments.
================
Comment at: llvm/lib/IR/AutoUpgrade.cpp:3857
void llvm::UpgradeARCRuntimeCalls(Module &M) {
+ // This lambda converts normal function calls to ARC runtime functions to
----------------
ahatanak wrote:
> steven_wu wrote:
> > I probably missed this from previous commit, shouldn't this function return `bool`?
> I changed the return type to void in a follow-up commit since nothing seemed to be using the return value. Do these functions have to return anything?
I guess it is just the convention and the return value is not used anymore. This is fine.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66047/new/
https://reviews.llvm.org/D66047
More information about the llvm-commits
mailing list