[PATCH] D105382: [GlobalISel] Tail call memcpy/memmove/memset even in the presence of copies

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 16 15:32:35 PDT 2021


jroelofs added a comment.

In D105382#2884618 <https://reviews.llvm.org/D105382#2884618>, @paquette wrote:

> So, this is always assuming we can treat G_MEMCPY and friends as `thisreturn`, right?

The patch this one depends on makes that guarantee.

> In D105671 <https://reviews.llvm.org/D105671>, @rjmccall mentioned that this may only be correct in terms of using it for ARC. Can we guarantee that we only do this for ARC here too?

The ARC thing was about doing the same for `objc_retain`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105382



More information about the llvm-commits mailing list