[PATCH] D105370: [GlobalISel] Mark memcpy/memmove/memset as thisreturn

[Jessica Paquette via Phabricator via llvm-commits]

paquette added a comment.

SDAG doesn't seem to mark the call as returned with the same example. Is this a new optimization?

https://clang.godbolt.org/z/K49z51zoo

IIRC `thisreturn` comes up as a result of the `returned` attribute in IR, right? Should the frontend be marking memory functions as `returned` instead? (I guess that we'd need a patch like this either way...?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105370