[PATCH] D70370: llvm/ObjCARC: Eliminate inlined AutoreleaseRV calls

Akira Hatanaka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 19:40:47 PST 2019


ahatanak added a comment.

I think I agree with you. `isSafeBetweenRVCalls` seems too conservative. I can't think of a case where we aren't allowed to remove the pair of instructions because the instructions you listed exist between the pair (and that includes opaque calls).  But John probably knows better whether that is safe or not.


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

https://reviews.llvm.org/D70370





More information about the llvm-commits mailing list