On Dec 20, 2012, at 5:47 PM, Jordan Rose <[email protected]> wrote:

> One comment for when this gets re-commited:
> 
>> 
>> +  llvm::ReversePostOrderTraversal<clang::CallGraph*> RPOT(&CG);
>> +  for (llvm::ReversePostOrderTraversal<clang::CallGraph*>::rpo_iterator
>> +         I = RPOT.begin(); I != RPOT.end(); ++I) {
> 
> Cache the end pointer here?
> 

Good catch. It was a copy and paste from the ReversePostOrderTraversal comment, 
so will change there as well.

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to