>
>
> https://codereview.chromium.org/14121004/diff/1/src/arm/lithium-codegen-arm.cc
>  
> File src/arm/lithium-codegen-arm.cc (right): 
>
> https://codereview.chromium.org/14121004/diff/1/src/arm/lithium-codegen-arm.cc#newcode2109
>  
>
> src/arm/lithium-codegen-arm.cc:2109<https://codereview.chromium.org/14121004/diff/1/src/arm/lithium-codegen-arm.cc#newcode2109src/arm/lithium-codegen-arm.cc:2109>:
>  
> if (!left_reg.is(result_reg)) { 
> Why do we need this "if" here? 
>
If left_reg is result_reg, "Move(result_reg, left_reg);" won't generate any 
code. So, it's no use to do a jump over nothing.

>
> https://codereview.chromium.org/14121004/ 
>

-- 
-- 
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.


Reply via email to