diff --git a/llvm/lib/Transforms/Utils/CodeExtractor.cpp b/llvm/lib/Transforms/Utils/CodeExtractor.cpp index 50eb8a27e07a..3d08a5dffc5f 100644 --- a/llvm/lib/Transforms/Utils/CodeExtractor.cpp +++ b/llvm/lib/Transforms/Utils/CodeExtractor.cpp @@ -67,6 +67,8 @@ static SetVector buildExtractionBlockSet(ArrayRef BBs) { SetVector Result; + assert(!BBs.empty()); + // Loop over the blocks, adding them to our set-vector, and aborting with an // empty set if we encounter invalid blocks. for (ArrayRef::iterator I = BBs.begin(), E = BBs.end(); @@ -80,6 +82,17 @@ buildExtractionBlockSet(ArrayRef BBs) { } } +#ifndef NDEBUG + for (ArrayRef::iterator I = llvm::next(BBs.begin()), + E = BBs.end(); + I != E; ++I) + for (pred_iterator PI = pred_begin(*I), PE = pred_end(*I); + PI != PE; ++PI) + assert(Result.count(*PI) && + "No blocks in this region may have entries from outside the region" + " except for the first block!"); +#endif + return Result; } @@ -665,15 +678,6 @@ Function *CodeExtractor::extractCodeRegion() { // block in the region. BasicBlock *header = *Blocks.begin(); - for (SetVector::iterator BI = llvm::next(Blocks.begin()), - BE = Blocks.end(); - BI != BE; ++BI) - for (pred_iterator PI = pred_begin(*BI), E = pred_end(*BI); - PI != E; ++PI) - assert(Blocks.count(*PI) && - "No blocks in this region may have entries from outside the region" - " except for the first block!"); - // If we have to split PHI nodes or the entry block, do so now. severSplitPHINodes(header);