Enable removing unreferenced labels for non-dbg as well

Meghana Gupta · Meghana Gupta · commit dd5209612bf8 · 2018-07-20T11:37:44.000-07:00
If the unreference label had a region, we will end up doing a lot of work in lowerer to generate returnthunks etc

Fixes OS#17415331
diff --git a/lib/Backend/FlowGraph.cpp b/lib/Backend/FlowGraph.cpp
@@ -1847,6 +1847,10 @@ FlowGraph::Destroy(void)
             }
         }
         NEXT_BLOCK;
+    }
+#endif
+    if (fHasTry)
+    {
         FOREACH_BLOCK_ALL(block, this)
         {
             if (block->GetFirstInstr()->IsLabelInstr())
@@ -1861,8 +1865,6 @@ FlowGraph::Destroy(void)
             }
         } NEXT_BLOCK;
     }
-#endif
-
     this->func->isFlowGraphValid = false;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1847,6 +1847,10 @@ FlowGraph::Destroy(void)`
`1847`	`1847`	`}`
`1848`	`1848`	`}`
`1849`	`1849`	`NEXT_BLOCK;`
	`1850`	`+ }`
	`1851`	`+#endif`
	`1852`	`+ if (fHasTry)`
	`1853`	`+ {`
`1850`	`1854`	`FOREACH_BLOCK_ALL(block, this)`
`1851`	`1855`	`{`
`1852`	`1856`	`if (block->GetFirstInstr()->IsLabelInstr())`
`@@ -1861,8 +1865,6 @@ FlowGraph::Destroy(void)`
`1861`	`1865`	`}`
`1862`	`1866`	`} NEXT_BLOCK;`
`1863`	`1867`	`}`
`1864`		`-#endif`
`1865`		`-`
`1866`	`1868`	`this->func->isFlowGraphValid = false;`
`1867`	`1869`	`}`
`1868`	`1870`