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

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 23:03:13 PST 2020


fedor.sergeev marked 2 inline comments 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)
----------------
asbirlea wrote:
> Conditionally getResult<MemDepAnalysis> in a follow up change?
Yep, will do as a separate one-liner after this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72732





More information about the llvm-commits mailing list