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

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith updated this revision to Diff 229920.
dexonsmith added a comment.

Respond to Akira's feedback.

- Use `isSafeBetweenRVCalls` instead of skipping past all opaque calls.
- Delete the other user of `isSafeBetweenRVCalls` in `ObjCARCOpt::OptimizeRetainRVCall`, since the new code already covers those cases.
- Check for PHI node equivalence.


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

https://reviews.llvm.org/D70370

Files:
  llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp
  llvm/test/Transforms/ObjCARC/inlined-autorelease-return-value.ll
  llvm/test/Transforms/ObjCARC/unsafe-claim-rv.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70370.229920.patch
Type: text/x-patch
Size: 20840 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191118/840afa26/attachment.bin>