1113367 - SIMD.int32x4(0, 0) gets TypeError

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Dan Gohman [:sunfish]]

Invoking the SIMD.int32x4 constructor with fewer than 4 arguments:

  SIMD.int32x4(0, 0)

gets this error:

  typein:1:0 TypeError: SIMD requires more than 3 arguments

This behavior differs from the polyfill, which takes any number of arguments and coerces 4 of them to integers with |0.

I don't currently have a strong opinion on which is right, although we do already have code in the ecmascript_simd testsuite which happens to do this, which is how I noticed this.

Comment 1

[:ProgramFOX]

Attachment: Proposed patch

Comment 2

[Benjamin Bouvier [:bbouvier] (inactive)]

Comment on attachment 8539679 [details] [diff] [review]
Proposed patch

Review of attachment 8539679 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for jumping in. However, I'm sorry the patch is merely a hack, plus it isn't generic (float32x4 have the same behavior). It is so trivial I'll just do it, but thanks anyways :)

Comment 3

[Benjamin Bouvier [:bbouvier] (inactive)]

Attachment: MozReview Request: bz://1113367/bbouvier

Comment 4

[Benjamin Bouvier [:bbouvier] (inactive)]

/r/1629 - Bug 1113367: SIMD (interpreter): make constructors arguments facultative;

Pull down this commit:

hg pull review -r daba7703abd9445dd9a2750383572845f7f48305

Comment 5

[Dan Gohman [:sunfish]]

https://reviewboard.mozilla.org/r/1629/#review1039

::: js/src/tests/ecma_7/SIMD/constructors.js
(Diff revision 1)
> +    assertEqX4(int32x4(1, 2, 3, 4), [1,2,3,4]);

Please add a few cases of constructors with more than 4 arguments too.

FYI, I filed bug https://github.com/johnmccutchan/ecmascript_simd/issues/102 on the topic of the special case of one argument.

Comment 6

[Dan Gohman [:sunfish]]

https://reviewboard.mozilla.org/r/1629/#review1041

Ship It!

Comment 7

[Dan Gohman [:sunfish]]

Comment on attachment 8540247 [details]
MozReview Request: bz://1113367/bbouvier

I assumed a "Ship It" meant r+, but I guess I shouldn't assume I know what to expect from MozReview yet. Here's the r+.

Comment 8

[Benjamin Bouvier [:bbouvier] (inactive)]

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

Comment 9

[Phil Ringnalda (:philor)]

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

Comment 11

[Benjamin Bouvier [:bbouvier] (inactive)]

Attachment: MozReview Request: Bug 1113367: SIMD (interpreter): make constructors arguments facultative;