http://codereview.chromium.org/2772004/diff/5001/6001 File src/arm/codegen-arm.cc (right):
http://codereview.chromium.org/2772004/diff/5001/6001#newcode9288 src/arm/codegen-arm.cc:9288: __ ubfx(r3, r0, 2, 1); // Extract bit 2 of r0 to bit 0 of r3. On 2010/06/11 14:26:15, Lasse Reichstein wrote:
Is ubfx supported on all platforms that we target? It seems to have been introduced in ARMv6T2, which is newer than the
instruction
list I have available.
No, so I have removed it. It was also not making a difference. These two lines have been removed from the CL. http://codereview.chromium.org/2772004/diff/5001/6001#newcode9289 src/arm/codegen-arm.cc:9289: __ tst(r3, r3); On 2010/06/11 14:26:15, Lasse Reichstein wrote:
Why are these two instructions better than the one move they replace?
They were not, so they have been removed. http://codereview.chromium.org/2772004/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
