ibmruntimes / v8ppc

Port of Google V8 javascript engine to PowerPC®
Other
94 stars 19 forks source link

mjsunit/regress/regress-1122.js triggers immediate field checking failure #33

Closed pengwuibm closed 11 years ago

pengwuibm commented 11 years ago

To reproduce, uncomment out NEW_IMM_CHECK_CODE in src/ppc/assembler-ppc.cc

Running on ppc native:

out/ppc.debug/d8 --test --nocrankshaft --nobreak-on-abort --nodead-code-elimination --enable-slow-asserts --debug-code --verify-heap test/mjsunit/mjsunit.js test/mjsunit/regress/regress-1122.js

val = 40012, 0x9c4c

#

Fatal error in ../src/ppc/assembler-ppc.cc, line 656

CHECK(is_int16(val)) failed

#

burdick commented 11 years ago

Updated to use new "form-aware" MacroAssembler methods and the test now passes. However there are many more instances of this same problem which we have to decide how to address.

burdick commented 11 years ago

Fixed with commit 604cfbec8d7cf5fc20fcb81e19c7c605af99dbf7