https://codereview.chromium.org/636283009/diff/100001/src/compiler/representation-change.h
File src/compiler/representation-change.h (right):
https://codereview.chromium.org/636283009/diff/100001/src/compiler/representation-change.h#newcode237
src/compiler/representation-change.h:237: op =
machine()->TruncateFloat64ToInt32();
The op is always the same in the end, probably no need to assign it
inside the conditional blocks.
https://codereview.chromium.org/636283009/diff/100001/src/compiler/typer.cc
File src/compiler/typer.cc (right):
https://codereview.chromium.org/636283009/diff/100001/src/compiler/typer.cc#newcode1090
src/compiler/typer.cc:1090: for (auto i =
typer_->weaken_min_limits_.begin();
range based for loops: for (const auto val : weaken_min_limits_) {...}
https://codereview.chromium.org/636283009/diff/100001/src/compiler/typer.cc#newcode1103
src/compiler/typer.cc:1103: for (auto i =
typer_->weaken_max_limits_.begin();
range based for loops: for (const auto val : weaken_max_limits_) {...}
https://codereview.chromium.org/636283009/diff/100001/test/cctest/types-fuzz.h
File test/cctest/types-fuzz.h (right):
https://codereview.chromium.org/636283009/diff/100001/test/cctest/types-fuzz.h#newcode106
test/cctest/types-fuzz.h:106: Integer =
Type::Range(isolate->factory()->NewNumber(-V8_INFINITY),
Shouldn't that be -inf..inf ?
https://codereview.chromium.org/636283009/
--
--
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.
