[PATCH] D41298: [Dominators] Remove verifyDomTree and add some verifying for Post Dom Trees

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 04:41:31 PST 2018


kuhar added inline comments.


================
Comment at: include/llvm/Support/GenericDomTree.h:322
     if (Roots.size() != Other.Roots.size())
-      return false;
+      return true;
 
----------------
dmgreen wrote:
> Can you check this is correct. This function, unusually, returns false on same, true on different.
Good catch! Feel free to fix it -- I think you can change this and the next one in a separate commit without review.


https://reviews.llvm.org/D41298





More information about the llvm-commits mailing list