1238236 - mandelbrot demo stops rendering if I turn on SIMD

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Mayank Bansal]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:46.0) Gecko/20100101 Firefox/46.0
Build ID: 20160108030334

Steps to reproduce:

1. start fresh profile
2. go to http://peterjensen.github.io/mandelbrot/js/mandelbrot-ww-asm.html
3. Start the demo.  You can see the demo rendered.

Now turn on SIMD.



Actual results:

with simd turned on, the demo doesnt render.


Expected results:

not so

Comment 1

[Mayank Bansal]

Attachment: aboutsupport.txt

Comment 2

[alex_mayorga]

¡Hola!

I can reproduce this on Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:46.0) Gecko/20100101 Firefox/46.0 ID:20160108030334 CSet: 1424cdfc075d1b7e277be914488ac73e20d1c982

This is what the Browser Console shows FWIW:

Use of getPreventDefault() is deprecated.  Use defaultPrevented instead. jquery-1.8.3.js:3255:28
TypeError: asm.js type error: bool32x4 is not a subtype of int32x4 mandelbrot-worker-asm.js:106:0
: Successfully compiled asm.js code (total compilation time 2ms; not stored in cache (too small to benefit)) mandelbrot-worker-asm.js

¡Gracias!

Comment 3

[Alice0775 White]

Regression window:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=1849d8dd63612df9b67712d81201d5cae09253c0&tochange=6d42293cc7917c59b2953de56a654a6a69d94867

Comment 4

[Jakob Stoklund Olesen [:jolesen]]

Should be fixed by https://github.com/PeterJensen/mandelbrot/pull/13

Bug 1160971 made an incompatible change to the SIMD.js functions when catching up to the latest version of the spec.

Ben, was that pull request ok to merge?

Comment 5

[Benjamin Bouvier [:bbouvier] (inactive)]

(In reply to Jakob Stoklund Olesen [:jolesen] from comment #4)
> Should be fixed by https://github.com/PeterJensen/mandelbrot/pull/13
> 
> Bug 1160971 made an incompatible change to the SIMD.js functions when
> catching up to the latest version of the spec.
> 
> Ben, was that pull request ok to merge?

Yes, thank you for the pull request, I just merged it, tested it and it compiles fine now.

Github doesn't send another email if there are commits added to a pull request, so I didn't see it was ready. Feel free to shout me an email or add a comment, next time :-)

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Looks like this was fixed in 46 back in January.