Now this feels nice and clean:) Maybe we should look into using ASTContext's parent map to replace ReverseAST in the future. LoopConvert predates the parent map so it could probably be improved. http://llvm-reviews.chandlerc.com/D950