[unittest] Remove TODO comment which caused concern
authorTobias Grosser <tobias@grosser.es>
Thu, 3 Aug 2017 04:17:58 +0000 (04:17 +0000)
committerTobias Grosser <tobias@grosser.es>
Thu, 3 Aug 2017 04:17:58 +0000 (04:17 +0000)
Remove the second part of the TODO comment that highlighted an issue with
possibly connecting all nodes to the exit of the CFG. This caused concerns
with Jakub Kuderski regarding its feasability, hence we remove it. Such
points are better discussed outside of CFG. If connecting all nodes makes
sense and what the impact is is currently part of an active review discussion.

llvm-svn: 309919

llvm/unittests/IR/DominatorTreeTest.cpp

index da5d502..ed8053e 100644 (file)
@@ -454,7 +454,7 @@ TEST(DominatorTree, DeletingEdgesIntroducesUnreachables) {
 // PDT. We also verify this property.
 //
 // TODO: Can we change the PDT definition such that C remains part of the
-//       CFG, at best without loosing the dominance relation D postdom B.
+//       CFG?
 TEST(DominatorTree, DeletingEdgesIntroducesInfiniteLoop) {
   StringRef ModuleString =
       "define void @f() {\n"