Reviewers: jarin, Benedikt Meurer,
https://codereview.chromium.org/1184183002/diff/1/BUILD.gn
File BUILD.gn (right):
https://codereview.chromium.org/1184183002/diff/1/BUILD.gn#newcode650
BUILD.gn:650: "src/compiler/greedy-allocator.h",
I am actually not confident this change needs to be done in this file.
The build was unblocked when I modified the gyp file in this change.
What is the purpose of this file?
Description:
Separated the new register allocator in its own files.
Opportunistically removed GreedyAllocator::TryReuseSpillForPhi because it is
actually unsuitable for Greedy. It was copied from Linear and it relies on
hints, however, the current implementation of hints assumes linear scan.
This change doesn't aim to address performance nor correctness for Greedy.
BUG=
Please review this at https://codereview.chromium.org/1184183002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+550, -591 lines):
M BUILD.gn
A src/compiler/greedy-allocator.h
A src/compiler/greedy-allocator.cc
M src/compiler/pipeline.cc
M src/compiler/register-allocator.h
M src/compiler/register-allocator.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.
