Closed
Bug 1212015
Opened 9 years ago
Closed 9 years ago
Fix a couple of OOM handling issues in AsmJS
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: jonco, Assigned: jonco)
Details
Attachments
(1 file, 1 obsolete file)
1.32 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #8670399 -
Flags: review?(terrence)
Comment 1•9 years ago
|
||
Comment on attachment 8670399 [details] [diff] [review] fix-asm-failure Review of attachment 8670399 [details] [diff] [review]: ----------------------------------------------------------------- Sorry for the drive-by review, I was curious about what was going wrong here... ::: js/src/asmjs/AsmJSValidate.cpp @@ +6401,5 @@ > } > > AsmFunction* asmFunc = lifo.new_<AsmFunction>(lifo); > + if (!asmFunc) > + return false; nice catch! @@ +6485,5 @@ > AsmModuleCompilerScope scope; > if (!CreateAsmModuleCompiler(m.compileInputs(), &scope)) > return false; > ModuleCompiler& mc = scope.module(); > + if (!mc.init()) CreateAsmModuleCompiler calls init() already. @@ +6662,5 @@ > if (!CreateAsmModuleCompiler(m.compileInputs(), &scope)) > return false; > ModuleCompiler& mc = scope.module(); > + if (!mc.init()) > + return false; ditto
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Benjamin Bouvier [:bbouvier] from comment #1) > CreateAsmModuleCompiler calls init() already. Oh, you fixed this yourself if bug 1181612, nice.
Assignee | ||
Comment 3•9 years ago
|
||
Even smaller patch this time.
Attachment #8670399 -
Attachment is obsolete: true
Attachment #8670399 -
Flags: review?(terrence)
Attachment #8670410 -
Flags: review?(terrence)
Comment 4•9 years ago
|
||
Comment on attachment 8670410 [details] [diff] [review] bug1212015-asmjs v2 Review of attachment 8670410 [details] [diff] [review]: ----------------------------------------------------------------- r="whatever Benjamin said"
Attachment #8670410 -
Flags: review?(terrence) → review+
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ddc9b0c250d3
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in
before you can comment on or make changes to this bug.
Description
•