https://chromiumcodereview.appspot.com/10007009/diff/1/src/profile-generator.h File src/profile-generator.h (right):
https://chromiumcodereview.appspot.com/10007009/diff/1/src/profile-generator.h#newcode1102 src/profile-generator.h:1102: void FillPostorderIndexes(Vector<HeapEntry*>* entries); On 2012/04/06 10:04:36, alexeif wrote:
On 2012/04/05 22:00:13, Mikhail Naganov (Chromium) wrote: > Why the rename? "Reverse" means that children will have order
indexes in
reverse > to their child index, as we are processing them backwards. This algorithm assigns postorder indexes which means the parent is
processed
after its nodes. AFAICT word "Reverse" came from the dominators
building typo: read "after its children".
algorithm where we're processing postorder sorted nodes in reverse
order. So the
word reverse has no relation to this particular function, thus I
removed it. https://chromiumcodereview.appspot.com/10007009/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
