The default bug view has changed. See this FAQ.

JS Correctness: Different output without -a and with -a

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: gkw, Unassigned)

Tracking

(Blocks: 2 bugs, {regression, testcase})

Trunk
x86_64
Linux
regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
function x(d) {} {
    x.__noSuchMethod__ = function() {}
}
try {
    print(x.eval())
} catch (e) {}

Shows an extra `undefined` error with & without -a.

$ ./js -m -a testcase.js
$ ./js -m testcase.js 
undefined

Tested on Ubuntu Linux 64-bit, m-c changeset 8ffdb4c7404a.

autoBisect shows this is probably related to the following changeset: (not sure if this is correct)

The first bad revision is:
changeset:   84036:7ab4f1ebc7cc
user:        Brian Hackett
date:        Mon Jan 09 06:29:50 2012 -0800
summary:     Backout 54cd89b0f1fa (bug 712714 backout).  Talos will probably report fake regressions for this patch, do not back out for this reason.
WFM on Mac x86.
(Reporter)

Comment 2

5 years ago
(In reply to Brian Hackett (:bhackett) from comment #1)
> WFM on Mac x86.

WFM on Mac x86/x64 too, but comment 0 was from Linux 64-bit. Retry on Linux?
(Reporter)

Comment 3

5 years ago
Likely fixed by bug 717208.

autoBisect shows this is probably related to the following changeset:

The first good revision is:
changeset:   84314:16f0d80b3137
user:        Brian Hackett
date:        Wed Jan 11 17:31:41 2012 -0800
summary:     Account for all special CALLPROP behavior in inline cache stub, bug 717208. r=dvander
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.