Closed Bug 1555153 Opened 5 years ago Closed 5 years ago

Differential Testing: Different output message involving division on ARM64

Categories

(Core :: JavaScript Engine: JIT, defect, P1)

ARM64
All
defect

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox-esr60 --- unaffected
firefox67 --- unaffected
firefox68 --- unaffected
firefox69 --- fixed

People

(Reporter: gkw, Assigned: nbp)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, testcase, Whiteboard: [fuzzblocker])

Attachments

(1 file)

x = [99999999999999999999999999, -(2 ** 53 - 2)];
for (let i = 0; i < 2; ++i) {
    print((x[i] | 0) / 3);
}
$ ./js-dbg-64-dm-armsim64-linux-x86_64-d8321d0b2c5b --fuzzing-safe --no-threads --ion-eager testcase.js
0
0
$ ./js-dbg-64-dm-armsim64-linux-x86_64-d8321d0b2c5b --fuzzing-safe --no-threads --baseline-eager --no-ion testcase.js
0
0.6666666666666666

Tested this on m-c rev d8321d0b2c5b.

My configure flags are:

AR=ar sh ./configure --enable-simulator=arm64 --enable-debug --enable-more-deterministic --with-ccache --enable-gczeal --enable-debug-symbols --disable-tests --disable-cranelift

python3 -u -m funfuzz.js.compile_shell -b "--enable-debug --enable-more-deterministic --enable-simulator=arm64" -r d8321d0b2c5b

Due to the simplicity of this bug, compare_jit is finding mostly dupes of this on ARM64.

autobisectjs shows this is probably related to the following changeset:

The first bad revision is:
changeset: https://hg.mozilla.org/mozilla-central/rev/7ed7b0b9f2dd
user: Nicolas B. Pierron
date: Mon May 27 16:36:26 2019 +0000
summary: Bug 1523568 - ARM64: Implement divisions by a constant. r=sstangl

Nicolas, is bug 1523568 a likely regressor?

Flags: needinfo?(nicolas.b.pierron)
Regressed by: 1523568

(In reply to Gary Kwong [:gkw] [:nth10sd] from comment #1)

Nicolas, is bug 1523568 a likely regressor?

Yes.

Assignee: nobody → nicolas.b.pierron
Status: NEW → ASSIGNED
Flags: needinfo?(nicolas.b.pierron)
Priority: -- → P1
Pushed by npierron@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/09088ff4d2a7 ARM64 division by constant: compare remainder against zero. r=sstangl
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: