1024786 - Assertion failure: windowLength <= windowRadius * 2, at frontend/TokenStream.cpp

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

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

Attachment: stack

The upcoming testcase asserts js debug shell on m-c changeset aab3362f97e9 with --ion-eager --ion-parallel-compile=off at Assertion failure: windowLength <= windowRadius * 2, at frontend/TokenStream.cpp

My configure flags are:

CC="clang -Qunused-arguments" CXX="clang++ -Qunused-arguments" AR=ar sh /Users/skywalker/trees/mozilla-central/js/src/configure --target=x86_64-apple-darwin12.5.0 --enable-optimize --enable-debug --enable-profiling --enable-gczeal --enable-debug-symbols --disable-tests --enable-more-deterministic --with-ccache --enable-threadsafe <other NSPR options>

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   https://hg.mozilla.org/mozilla-central/rev/43acd23f5a98
user:        Brian Hackett
date:        Thu May 15 16:48:21 2014 -0700
summary:     Bug 976446 - Add port of irregexp regexp engine, and use by default, r=jandem.

Brian, is bug 976446 a likely regressor?

Comment 1

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

Attachment: testcase

$ ./js-dbg-64-dm-ts-darwin-aab3362f97e9 --ion-eager --ion-parallel-compile=off testcase.js
Assertion failure: windowLength <= windowRadius * 2, at /Users/skywalker/trees/mozilla-central/js/src/frontend/TokenStream.cpp:683
Segmentation fault: 11

Comment 2

[Brian Hackett [Laid off!]]

Attachment: patch

Computing a caller filename when one wasn't provided confused the code that prints leading/trailing context for error messages.

Comment 3

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

Please land this testcase as well - I recall this one was a pain to reduce. Thanks! :)

Comment 4

[Brian Hackett [Laid off!]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/de4ce465c58c

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/de4ce465c58c