Please rebase this onto https://codereview.chromium.org/696363002/ after I
landed it, then this CL should essentially turn into a one-line change.
https://codereview.chromium.org/686273005/diff/1/src/compiler/scheduler.cc
File src/compiler/scheduler.cc (right):
https://codereview.chromium.org/686273005/diff/1/src/compiler/scheduler.cc#newcode535
src/compiler/scheduler.cc:535: void ComputeAssemblyOrder() {
There is a call to this method missing for the normal scheduler
pipeline, the two calls below only cover the side-channel for the raw
machine assembler and the floating control fusion.
https://codereview.chromium.org/686273005/diff/1/src/compiler/scheduler.cc#newcode769
src/compiler/scheduler.cc:769:
l->block->set_ao_number(static_cast<int>(final_order->size()));
This gives you the RPO numbers, not the AO numbers.
https://codereview.chromium.org/686273005/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
