]> granicus.if.org Git - llvm/commitdiff
[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.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@309919 91177308-0d34-0410-b5e6-96231b3b80d8

unittests/IR/DominatorTreeTest.cpp

index da5d5025e23e30545aeded68734c263a823e9782..ed8053ef5abb7d56b055a195df6af7b3244b7d2a 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"