Last Comment Bug 718076 - JS Correctness: Different output without -a and with -a
: JS Correctness: Different output without -a and with -a
Status: RESOLVED FIXED
: regression, testcase
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: ---
Assigned To: general
:
Mentors:
Depends on:
Blocks: jsfunfuzz js-differential-test 712714
  Show dependency treegraph
 
Reported: 2012-01-13 14:39 PST by Gary Kwong [:gkw] [:nth10sd]
Modified: 2012-06-27 13:57 PDT (History)
5 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments

Description Gary Kwong [:gkw] [:nth10sd] 2012-01-13 14:39:09 PST
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.
Comment 1 Brian Hackett (:bhackett) 2012-01-13 14:48:41 PST
WFM on Mac x86.
Comment 2 Gary Kwong [:gkw] [:nth10sd] 2012-01-13 14:56:04 PST
(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?
Comment 3 Gary Kwong [:gkw] [:nth10sd] 2012-06-27 13:57:25 PDT
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

Note You need to log in before you can comment on or make changes to this bug.