1190002 - Assertion failure: HasSSE2(), at jit/x86-shared/Assembler-x86-shared.h

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

// Randomly chosen test: js/src/jit-test/tests/ion/lsra-bug1112164.js
function f() {
    return [Math.tan(1, 1)];
}
for (var x = 0; x < 99; x++) {
    f();
}

asserts js debug shell on m-c changeset afa67b6957bb with --fuzzing-safe --no-threads --ion-eager --unboxed-arrays --no-fpu at Assertion failure: HasSSE2(), at jit/x86-shared/Assembler-x86-shared.h

Configure options:

LD=ld CROSS_COMPILE=1 CC="clang -Qunused-arguments -msse2 -mfpmath=sse -arch i386" RANLIB=ranlib CXX="clang++ -Qunused-arguments -msse2 -mfpmath=sse -arch i386" AS=$CC AR=ar STRIP="strip -x -S" HOST_CC="clang -Qunused-arguments -msse2 -mfpmath=sse" AUTOCONF=/usr/local/Cellar/autoconf213/2.13/bin/autoconf213 HOST_CXX="clang++ -Qunused-arguments -msse2 -mfpmath=sse" sh /Users/skywalker/trees/mozilla-central/js/src/configure --target=i386-apple-darwin9.2.0 --enable-macos-target=10.5 --enable-debug --enable-nspr-build --enable-more-deterministic --with-ccache --enable-gczeal --enable-debug-symbols --disable-tests

python -u ~/funfuzz/js/compileShell.py -b "--enable-debug --enable-more-deterministic --enable-nspr-build --32" -r afa67b6957bb

autoBisect is running.

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Attachment: stack

(lldb) bt 5
* thread #1: tid = 0x9456c, 0x00862da6 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`js::jit::AssemblerX86Shared::vxorpd(this=<unavailable>, src1=(reg_ = xmm7, type_ = Double, isInvalid_ = false), src0=(reg_ = xmm7, type_ = Double, isInvalid_ = false), dest=(reg_ = xmm7, type_ = Double, isInvalid_ = false)) + 198 at Assembler-x86-shared.h:2741, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x00862da6 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`js::jit::AssemblerX86Shared::vxorpd(this=<unavailable>, src1=(reg_ = xmm7, type_ = Double, isInvalid_ = false), src0=(reg_ = xmm7, type_ = Double, isInvalid_ = false), dest=(reg_ = xmm7, type_ = Double, isInvalid_ = false)) + 198 at Assembler-x86-shared.h:2741
    frame #1: 0x007511b8 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`void js::jit::MacroAssembler::storeUnboxedProperty<js::jit::BaseIndex>(js::jit::BaseIndex, JSValueType, js::jit::ConstantOrRegister, js::jit::Label*) [inlined] js::jit::MacroAssemblerX86Shared::zeroDouble(js::jit::FloatRegister) + 1512 at MacroAssembler-x86-shared.h:904
    frame #2: 0x00751186 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`void js::jit::MacroAssembler::storeUnboxedProperty<js::jit::BaseIndex>(js::jit::BaseIndex, JSValueType, js::jit::ConstantOrRegister, js::jit::Label*) [inlined] js::jit::MacroAssemblerX86Shared::convertInt32ToDouble(js::jit::Register, js::jit::FloatRegister) at MacroAssembler-x86-shared.h:656
    frame #3: 0x00751186 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`void js::jit::MacroAssembler::storeUnboxedProperty<js::jit::BaseIndex>(js::jit::BaseIndex, JSValueType, js::jit::ConstantOrRegister, js::jit::Label*) [inlined] js::jit::MacroAssemblerX86::int32ValueToDouble(js::jit::ValueOperand const&, js::jit::FloatRegister) at MacroAssembler-x86.h:976
    frame #4: 0x00751186 js-dbg-32-dm-nsprBuild-darwin-afa67b6957bb`void js::jit::MacroAssembler::storeUnboxedProperty<js::jit::BaseIndex>(this=0xbfffe2a8, address=BaseIndex at 0xbfffe0d4, type=JSVAL_TYPE_DOUBLE, value=ConstantOrRegister at 0xbfffe0e8, failure=<unavailable>) + 1462 at MacroAssembler.cpp:961
(lldb)

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   https://hg.mozilla.org/mozilla-central/rev/57dce88fc620
user:        Brian Hackett
date:        Tue May 26 16:29:19 2015 -0600
summary:     Bug 1165392, Bug 1165463 - Various unboxed array fixes and optimizations, r=jandem.

Brian, is bug 1165392 or bug 1165463 a likely regressor?

Comment 3

[Brian Hackett [Laid off!]]

Attachment: patch

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1a410d6cb2ba

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1a410d6cb2ba