[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 10:32:39 PDT 2019


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
----------------
I probably missed this from previous commit, shouldn't this function return `bool`?


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