diff options
| author | Tobias Grosser <tobias@grosser.es> | 2017-08-03 04:17:58 +0000 |
|---|---|---|
| committer | Tobias Grosser <tobias@grosser.es> | 2017-08-03 04:17:58 +0000 |
| commit | bd351effcdb5cf55b72defcdccad905baf1f2d52 (patch) | |
| tree | 2cbc597d3f5b2270f4b3bc19be32b4a13eea3bd5 /llvm/unittests/IR/DominatorTreeTest.cpp | |
| parent | 8c549de7617502f90ac3738690e406dd42fd35f8 (diff) | |
| download | bcm5719-llvm-bd351effcdb5cf55b72defcdccad905baf1f2d52.tar.gz bcm5719-llvm-bd351effcdb5cf55b72defcdccad905baf1f2d52.zip | |
[unittest] Remove TODO comment which caused concern
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
Diffstat (limited to 'llvm/unittests/IR/DominatorTreeTest.cpp')
| -rw-r--r-- | llvm/unittests/IR/DominatorTreeTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/unittests/IR/DominatorTreeTest.cpp b/llvm/unittests/IR/DominatorTreeTest.cpp index da5d5025e23..ed8053ef5ab 100644 --- a/llvm/unittests/IR/DominatorTreeTest.cpp +++ b/llvm/unittests/IR/DominatorTreeTest.cpp @@ -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" |
