http://codereview.chromium.org/6197003/diff/1/src/arm/macro-assembler-arm.cc File src/arm/macro-assembler-arm.cc (right):
http://codereview.chromium.org/6197003/diff/1/src/arm/macro-assembler-arm.cc#newcode552 src/arm/macro-assembler-arm.cc:552: vmrs(fpscr_flags); On 2011/01/11 23:42:26, Rodolph Perfetta wrote:
I know this is not part of the change, but the vmrs here should be
conditional.
Otherwise LGTM.
Thanks for spotting this - it must have slipped from my previous commit. http://codereview.chromium.org/6197003/diff/1/src/arm/macro-assembler-arm.cc#newcode561 src/arm/macro-assembler-arm.cc:561: vmrs(fpscr_flags); On 2011/01/11 23:42:26, Rodolph Perfetta wrote:
Ditto.
Done. http://codereview.chromium.org/6197003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
