Closed
Bug 940061
Opened 11 years ago
Closed 11 years ago
Fix JS_STACK_GROWTH_DIRECTION > 0
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: jer, Assigned: aaro.koskinen)
References
Details
(Whiteboard: [bugday-20131125])
Attachments
(2 files)
269 bytes,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
746 bytes,
patch
|
billm
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux parisc64; rv:25.0) Gecko/20100101 Firefox/25.0 (Beta/Release)
Build ID: 20131118203615
Steps to reproduce:
On HPPA (PA RISC) and probably IA64 (Itanium), the stack grows up, not down. The JS code supports this, but recent changes (since version 20) broke that support.
Actual results:
Compile failure.
Expected results:
With the attached patch, Firefox should compile nicely on platforms where the stack grows up, not down. Writing this in Firefox on just such a platform (Gentoo Linux/HPPA).
Reporter | ||
Comment 1•11 years ago
|
||
Another JS_STACK_GROWTH_DIRECTION patch Gentoo has been carrying for a while.
![]() |
||
Updated•11 years ago
|
Component: Untriaged → JavaScript Engine
Product: Firefox → Core
Whiteboard: [bugday-20131125]
Comment 2•11 years ago
|
||
Comment on attachment 8334171 [details] [diff] [review]
7006_fixup_hppa_support.patch
Bill, hg says you're the last one who modified that line :)
Attachment #8334171 -
Flags: review?(wmccloskey)
Comment 3•11 years ago
|
||
Comment on attachment 8334144 [details] [diff] [review]
firefox-25.0-stack-grows-up.patch
Luke, hg says you're the last one who modified that line :)
Attachment #8334144 -
Flags: review?(luke)
![]() |
||
Updated•11 years ago
|
Attachment #8334144 -
Flags: review?(luke) → review+
Attachment #8334171 -
Flags: review?(wmccloskey) → review+
Comment 4•11 years ago
|
||
Should be noted that Attachment #8334171 [details] [diff] has landed on m-c , unsure when.
Comment 5•11 years ago
|
||
as i said these are already landed; no point leaving this open.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•