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

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 23 11:56:50 PDT 2021


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



More information about the llvm-commits mailing list