http://codereview.chromium.org/6730029/diff/1/src/arm/lithium-codegen-arm.cc File src/arm/lithium-codegen-arm.cc (right):
http://codereview.chromium.org/6730029/diff/1/src/arm/lithium-codegen-arm.cc#newcode1178 src/arm/lithium-codegen-arm.cc:1178: bool can_overflow = instr->hydrogen()->CheckFlag(HValue::kCanOverflow); On 2011/03/24 20:32:24, Søren Gjesse wrote:
Can't we just use SetCC always?
We could, but on modern cores, setting the flags when not needed introduces a dependency which tend to reduce the core efficiency. Admittedly the impact for one instruction is going to be very small. If you want I'll revert that. http://codereview.chromium.org/6730029/diff/1/src/arm/lithium-codegen-arm.cc#newcode1278 src/arm/lithium-codegen-arm.cc:1278: bool can_overflow = instr->hydrogen()->CheckFlag(HValue::kCanOverflow); On 2011/03/24 20:32:24, Søren Gjesse wrote:
Ditto.
see previous answer. http://codereview.chromium.org/6730029/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
