[PATCH] D55120: [MemCpyOpt] memset->memcpy forwarding with undef tail

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: lib/Analysis/MemoryDependenceAnalysis.cpp:173
+      Loc = MemoryLocation::getForArgument(II, 0, TLI);
+      return ModRefInfo::Mod;
     default:
----------------
This probably makes sense, but it looks like it'll impact other transforms; would it be possible to test separately?  Or does it actually not have any impact on other transforms for some reason?

Looks fine otherwise.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55120