Hi Cameron, > + while (!VisitStack.empty()) { > + BasicBlock *BB = VisitStack.back(); > + VisitStack.pop_back(); the above two lines can be replaced with one using pop_back_val. Ciao, Duncan.