Reviewers: ,
Message:
Deferred block splitting as a stand-alone stage, running independent of
register
allocator.
https://codereview.chromium.org/1271703002/diff/1/src/compiler/register-allocator.cc
File src/compiler/register-allocator.cc (right):
https://codereview.chromium.org/1271703002/diff/1/src/compiler/register-allocator.cc#newcode1596
src/compiler/register-allocator.cc:1596: Define(curr_position, output);
I should be able to write a unit test for this one. It was caught by
some tests, when I introduced the deferred block splitting phase.
https://codereview.chromium.org/1271703002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):
https://codereview.chromium.org/1271703002/diff/1/src/flag-definitions.h#newcode402
src/flag-definitions.h:402: DEFINE_IMPLICATION(turbo_greedy_regalloc,
turbo_preprocess_ranges)
The implication is only so we can get trybot coverage with the greedy
trybots, it can be removed if we have dedicated trybots for the
preprocessing feature.
Description:
Stand-alone deferred block splitting.
BUG=
patch from issue 1265103002 at patchset 20001
(http://crrev.com/1265103002#ps20001)
Please review this at https://codereview.chromium.org/1271703002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+584, -19 lines):
M BUILD.gn
M src/compiler/instruction.h
M src/compiler/move-optimizer.cc
M src/compiler/pipeline.cc
A src/compiler/preprocess-live-ranges.h
A src/compiler/preprocess-live-ranges.cc
M src/compiler/register-allocator.h
M src/compiler/register-allocator.cc
M src/flag-definitions.h
M test/unittests/compiler/instruction-sequence-unittest.h
M test/unittests/compiler/instruction-sequence-unittest.cc
M test/unittests/compiler/register-allocator-unittest.cc
M tools/gyp/v8.gyp
--
--
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.
