[PATCH] D119929: [MemCpyOpt] Check uses of found Clobber in writtenBetween.
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 21 07:21:38 PST 2022
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Transforms/Scalar/MemCpyOptimizer.cpp:360
MemoryAccess *Clobber = MSSA->getWalker()->getClobberingMemoryAccess(
End->getDefiningAccess(), Loc);
+
----------------
I'd move the getClobberingMemoryAccess() call into the `!isa<MemoryUse>` branch. Not much point doing it if we're going to discard it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119929/new/
https://reviews.llvm.org/D119929
More information about the llvm-commits
mailing list