https://codereview.chromium.org/21505002/diff/3001/src/hydrogen.cc
File src/hydrogen.cc (right):
https://codereview.chromium.org/21505002/diff/3001/src/hydrogen.cc#newcode7731
src/hydrogen.cc:7731: HValue* context) {
On 2013/08/01 09:21:09, Sven Panne wrote:
Hmmm, having 8 arguments is not very nice, but let's see how it will
be used
elsewhere and perhaps refactor that. A TODO would be nice.
well i know that the stub will have to pass exactly those args, since in
the case of stybs the type-feedback comes directly from the
extra-ic-state and not the Type system....
https://codereview.chromium.org/21505002/diff/3001/src/hydrogen.cc#newcode7736
src/hydrogen.cc:7736: if (op != Token::ADD ||
On 2013/08/01 09:21:09, Sven Panne wrote:
Brain explosion caused by heavy use of negation... o_O Pushing the
negation
outwards, things are much clearer:
if (!(op == Token::ADD &&
(left_type->Maybe(Type::String()) ||
right_type->Maybe(Type::String())))) {
Naming the expression would be even better:
bool maybe_string_addition = ...
if (!maybe_string_addition) {
This would even make the comment in the if body obsolete.
"Comments are a sign of our inability to express things in code." :-)
yup, mechanical refactoring issue...
https://codereview.chromium.org/21505002/
--
--
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/groups/opt_out.
