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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:2725
     AU.addRequired<LoopInfoWrapperPass>();
     if (!NoMemDepAnalysis)
       AU.addRequired<MemoryDependenceWrapperPass>();
----------------
This could use Impl. isMemDepEnabled as well, right? Then we could get rid of the NoMemDepAnalysis variable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72732