On 10/17/2012 10:01 PM, Nicholas Nethercote wrote: > On Thu, Oct 18, 2012 at 1:29 PM, Brendan Eich <[email protected]> wrote: >> $ ./Darwin_DBG.OBJ/js -f tests/shell.js -f tests/js1_8/shell.js -f >> tests/js1_8/genexps/shell.js -f tests/js1_8/genexps/regress-380237-01.js - >> BUGNUMBER: 380237 >> STATUS: Generator expressions - sudoku >> 4 8 3 |9 2 1 |6 5 7 >> 9 6 7 |3 4 5 |8 2 1 >> 2 5 1 |8 7 6 |4 9 3 >> ------+------+------ >> 5 4 8 |1 3 2 |9 7 6 >> 7 2 9 |5 6 4 |1 3 8 >> 1 3 6 |7 9 8 |2 4 5 >> ------+------+------ >> 3 7 2 |6 8 9 |5 1 4 >> 8 1 4 |2 5 3 |7 6 9 >> 6 9 5 |4 1 7 |3 8 2 >> >> >> tests/js1_8/genexps/regress-380237-01.js:131:0 InternalError: too much >> recursion > > Terrence and I have had problems with that one before. IIRC the test > has some bogosity about it, but I don't remember the details -- > Terrence, do you?
I'll just echo what Nicolas said: anything that changes the interpreter stack will randomly make this test explode in this way. That said, I also hit this a bunch when implementing the StoreBuffer and that code explicitly goes nowhere near the stack. I vaguely remember that someone looked into it and found their compiler randomly deciding to give js::Interpret a bogusly humongous frame. I was /really/ hoping that unraveling the threaded interpreter would make this problem go away permanently. That work landed over a week ago in 109802:44079242ee9b. Brendan, what revision are you testing? -Terrence > Nick > _______________________________________________ > dev-tech-js-engine-internals mailing list > [email protected] > https://lists.mozilla.org/listinfo/dev-tech-js-engine-internals _______________________________________________ dev-tech-js-engine-internals mailing list [email protected] https://lists.mozilla.org/listinfo/dev-tech-js-engine-internals
