serge-sans-paille added a comment.

In D71374#1783245 <https://reviews.llvm.org/D71374#1783245>, @xbolva00 wrote:

> LLVM already converts mempcpy to memcpy..


Indeed, the clang version I was using as base reference was clang-9, and the 
mempcpy optimisation at IR level got introduced after that.
Nevertheless, this patch does much more than lowering mempcpy, it also triggers 
new warnings, so It's still useful that clang understands it.

@xbolva00 should I remove the lowering part and leave it to llvm?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71374



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to