Closed
Bug 1482294
Opened 6 years ago
Closed 6 years ago
disable jit unwind handling on aarch64 windows
Categories
(Core :: JavaScript Engine: JIT, enhancement)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: froydnj, Assigned: froydnj)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
As the comment for SetJitExceptionHandler makes clear, the
infrastructure we have for generating unwind information on 64-bit
Windows is only necessary to permit Breakpad to generate crash reports.
We don't even have crash reporting for our non-existent AArch64 Windows
builds, and it will likely take us some time to make the necessary
changes in Breakpad and elsewhere. In addition, the unwind information
format is completely different on AArch64, and there's no decent
documentation on it yet.
Given all of this, the easiest way forward right now is to simply
disable this code to get things compiling. We can reenable it later
once we understand how to add appropriate support.
Assignee | ||
Comment 1•6 years ago
|
||
I don't particularly like doing this, but I'm open to better options.
Attachment #8999042 -
Flags: review?(jdemooij)
Assignee | ||
Updated•6 years ago
|
Blocks: aarch64-windows
Comment 2•6 years ago
|
||
Comment on attachment 8999042 [details] [diff] [review]
disable jit unwind handling on aarch64 windows
Seems reasonable. Maybe file a follow-up bug blocking the relevant ship-on-windows-aarch64 bug just to make sure we double check this eventually?
Attachment #8999042 -
Flags: review?(jdemooij) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/81466097f47d
disable jit unwind handling on aarch64 windows; r=jandem
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•