Addressed feedback.
https://codereview.chromium.org/1311983002/diff/80001/src/compiler/register-allocator.cc
File src/compiler/register-allocator.cc (right):
https://codereview.chromium.org/1311983002/diff/80001/src/compiler/register-allocator.cc#newcode2027
src/compiler/register-allocator.cc:2027: auto range =
data()->GetOrCreateLiveRangeFor(operand_index);
On 2015/08/26 04:31:20, Benedikt Meurer wrote:
Nit: please deautofy this.
Done.
https://codereview.chromium.org/1311983002/diff/80001/src/compiler/register-allocator.h
File src/compiler/register-allocator.h (right):
https://codereview.chromium.org/1311983002/diff/80001/src/compiler/register-allocator.h#newcode285
src/compiler/register-allocator.h:285: TopLevelLiveRange* TopLevel() {
return parent_; }
On 2015/08/26 04:31:20, Benedikt Meurer wrote:
parent_ should be renamed to toplevel_ (or top_level_ to keep the
naming in
sync).
Done.
https://codereview.chromium.org/1311983002/diff/80001/src/compiler/register-allocator.h#newcode289
src/compiler/register-allocator.h:289: return
reinterpret_cast<LiveRange*>(parent_) == this;
On 2015/08/26 04:31:20, Benedikt Meurer wrote:
The reinterpret_cast is unnecessary (and dangerous) here. Just delay
the
definition of this function until after the definition of
TopLevelLiveRange and
everything will be fine.
Done.
https://codereview.chromium.org/1311983002/
--
--
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.
