[PATCH] D104801: [MemCpyOpt] Enable memcpy optimization for NVPTX back-end.

[Artem Belevich via Phabricator via llvm-commits]

tra added a comment.

In D104801#2836730 <https://reviews.llvm.org/D104801#2836730>, @nikic wrote:

> MemCpyOpt works on intrinsics, not libcalls. Why is it checking TLI at all?
>
> I think the only place TLI should come in is when SimplifyLibCall converts memcpy into llvm.memcpy, but MemCpyOpt itself shouldn't care about it.

The check has been added long ago to deal with -fno-builtin, but it's not clear what exactly was the issue.
https://github.com/llvm/llvm-project/commit/23f61a09aff4a3c5ca4bba9410878dcfebf0656c


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104801