[PATCH] D118443: Add support for atomic memory copy lowering

[Dawid Jurczak via Phabricator via llvm-commits]

yurai007 added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LowerMemIntrinsics.cpp:513
+template<typename T>
+static bool canOverlap(MemTransferBase<T> *Memcpy, ScalarEvolution *SE) {
   if (SE) {
----------------
nit: is there any reason for templating first parameter? For now we only pass concrete MemCpyInst as argument.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118443