[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

Akira Hatanaka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 9 10:05:09 PST 2021


ahatanak added a comment.

It looks like there is a pre-existing bug in `replaceUsesOfNonProtoConstant` where the operand bundles of all call sites are accumulated into `newBundles`. This will be fixed if its declaration is moved into the loop body. Also, we found another case of deadargelim changing the return type of the called function to `void`. I'll post a new patch when I have the fixes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92808



More information about the cfe-commits mailing list