[PATCH] D42337: [Dominators] Introduce DomTree verification levels

[Brian Rzycki via Phabricator via llvm-commits]

brzycki added a comment.

LGTM. I just have one question about the patch, see inline.



================
Comment at: include/llvm/Support/GenericDomTree.h:775
+  ///             as tree construction).
+  bool verify(VerificationLevel VL = VerificationLevel::Full) const {
+    return DomTreeBuilder::Verify(*this, VL);
----------------
Is the Full level the same as what verify did before this patch? I'm curious how the default level was chosen.


Repository:
  rL LLVM

https://reviews.llvm.org/D42337