[Lldb-commits] [lldb] r106116 - in /lldb/trunk: include/lldb/Symbol/Symtab.h source/Symbol/Symtab.cpp

Chris Lattner clattner at apple.com
Wed Jun 16 17:46:28 PDT 2010


On Jun 16, 2010, at 10:34 AM, Owen Anderson wrote:

> +    // Sort the indexes in place using std::sort
> +    std::stable_sort(indexes.begin(), indexes.end(), SymbolIndexComparator(m_symbols));

Hi Owen,

Please correct the comment.  Also, please expand it to talk about why stable_sort is the right answer here (it's a perf win, not important for correctness).

Thanks!

-Chris





More information about the lldb-commits mailing list