861705 - auto-regress/bug755564.js and auto-regress/bug783923.js and eight more fail when parallelarray is disabled

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Phil Ringnalda (:philor)]

https://tbpl.mozilla.org/?tree=Try&rev=0eaac9f574d7 is current-Aurora pushed to Try as though it had already been merged to Beta, showing the jit-test failures we'll see when it does get merged, because auto-regress/bug755564.js and auto-regress/bug783923.js run even when getBuildConfiguration().parallelJS is false.

Comment 1

[Lukas Blakk [:lsblakk] use ?needinfo]

Tracking since this looks to impact our upcoming Aurora->Beta merge.

Comment 2

[Niko Matsakis [:nmatsakis]]

Attachment: Check whether ParallelArray is disabled

Comment 3

[Phil Ringnalda (:philor)]

Oops, apparently the "Only displaying first 20 of 100+ failures" text on tbpl isn't big enough to actually notice, so I missed seeing that the full list of failures in https://tbpl.mozilla.org/php/getParsedLog.php?id=22075889 is more than just those two.

Comment 4

[Phil Ringnalda (:philor)]

(And also that the list at the top of the log is only "First 100 of 122 failures" so it's not two, and not two plus eight more either.)

Comment 5

[Niko Matsakis [:nmatsakis]]

Attachment: Check whether ParallelArray is disabled

Extend same approach as attachment 739076 [details] [diff] [review] but to more tests. Carrying over r+ from sstangl.

Comment 6

[Niko Matsakis [:nmatsakis]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/658d81e9c17f

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/658d81e9c17f

Comment 8

[Phil Ringnalda (:philor)]

https://tbpl.mozilla.org/?tree=Try&rev=9e9442ddaed1 is aurora-as-beta liking that patch (well, the parts of it that apply) just fine, land that a=test-only sucker!

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/68829788567a