[PATCH] D72732: [GVN] introduce GVNOptions to control GVN pass behavior

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev marked an inline comment as done.
fedor.sergeev added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:636
+  bool Changed = runImpl(F, AC, DT, TLI, AA,
+                         isMemDepEnabled() ? &MemDep : nullptr, LI, &ORE);
   if (!Changed)
----------------
fedor.sergeev wrote:
> asbirlea wrote:
> > Conditionally getResult<MemDepAnalysis> in a follow up change?
> Yep, will do as a separate one-liner after this patch.
since this patch became rather involved on legacy part anyway, 
I decided that doing this change here (and not in a followup) is fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72732