More Dehydra/Spidermonkey problems with hasOwnProperty

RESOLVED FIXED

Status

Firefox Build System
Source Code Analysis
RESOLVED FIXED
8 years ago
5 months ago

People

(Reporter: Ehren Metcalfe, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 1 obsolete attachment)

2.92 KB, text/plain
Details
168 bytes, application/x-javascript
Details
16 bytes, text/plain
Details
622 bytes, patch
Ehren Metcalfe
: review+
Details | Diff | Splinter Review
(Reporter)

Description

8 years ago
Created attachment 449061 [details]
backtrace

I've been running SpiderMonkey from rev. 89e665eb9944 to avoid the test-suite failures in bug 526299 however I've just encountered a similar problem. Not sure if this is a SpiderMonkey issue or a "we're embedding SpiderMonkey wrong" issue.

Output:

/home/ehren/gcc4.5/dist.4.5.0/bin/g++ has.c -fplugin=/home/ehren/gcc4.5/dehydra/gcc_treehydra.so -fplugin-arg-gcc_treehydra-script=/home/ehren/dehydra-tests/testcode/../hasOwn.js 
/home/ehren/gcc4.5/dehydra/libs/treehydra.js:33: JS Exception: No blah in this lazy object
:0:     #0: Error("No blah in this lazy object")
/home/ehren/gcc4.5/dehydra/libs/treehydra.js:33:      #1: unhandledLazyProperty("blah")
/home/ehren/dehydra-tests/testcode/../hasOwn.js:7:        #2: process_tree([object GCCNode])
make: *** [blahmake] Error 1

(See attached backtrace and testcase)
(Reporter)

Comment 1

8 years ago
Created attachment 449062 [details]
treehydra script
(Reporter)

Comment 2

8 years ago
Created attachment 449063 [details]
test code

Updated

8 years ago
Depends on: 570195
(Reporter)

Comment 3

8 years ago
Created attachment 449340 [details] [diff] [review]
for testsuite

can confirm that patch in bug 570195 fixes this.
Attachment #449340 - Flags: review?(tglek)

Updated

8 years ago
Attachment #449340 - Flags: review?(tglek) → review+

Comment 4

8 years ago
Comment on attachment 449340 [details] [diff] [review]
for testsuite

Should put the spidermonkey bugno in a comment in that file in case people run into it.
(Reporter)

Comment 5

8 years ago
Created attachment 449342 [details] [diff] [review]
for testsuite

carrying over r+
Attachment #449340 - Attachment is obsolete: true
Attachment #449342 - Flags: review+
(Reporter)

Comment 6

8 years ago
http://hg.mozilla.org/rewriting-and-analysis/dehydra/rev/225275b2e174
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Updated

5 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.