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

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

Thanks, this looks useful!



================
Comment at: llvm/include/llvm/Transforms/Scalar/GVN.h:90
+
+  /// Enables or disables PRE of loads in GVN.
+  GVNOptions &setMemDep(bool MemDep) {
----------------
Update comment?


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:636
+  bool Changed = runImpl(F, AC, DT, TLI, AA,
+                         isMemDepEnabled() ? &MemDep : nullptr, LI, &ORE);
   if (!Changed)
----------------
Conditionally getResult<MemDepAnalysis> in a follow up change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72732