llvm.org GIT mirror llvm / 98bb5a4
[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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@309919 91177308-0d34-0410-b5e6-96231b3b80d8 Tobias Grosser 2 years ago
1 changed file(s) with 1 addition(s) and 1 deletion(s). Raw diff Collapse all Expand all
453453 // PDT. We also verify this property.
454454 //
455455 // TODO: Can we change the PDT definition such that C remains part of the
456 // CFG, at best without loosing the dominance relation D postdom B.
456 // CFG?
457457 TEST(DominatorTree, DeletingEdgesIntroducesInfiniteLoop) {
458458 StringRef ModuleString =
459459 "define void @f() {\n"