Closed
Bug 1170596
Opened 11 years ago
Closed 11 years ago
IonGraph not working anymore on functions which contain infinite loops
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla41
| Tracking | Status | |
|---|---|---|
| firefox41 | --- | fixed |
People
(Reporter: bbouvier, Assigned: nbp)
Details
Attachments
(1 file, 1 obsolete file)
|
1004 bytes,
patch
|
bbouvier
:
review+
|
Details | Diff | Splinter Review |
I've tried both IONFLAGS=logs, and IONFLAGS=logs-sync with --ion-offthread-compile=off, without success. The smallest test case I have is a function which contains an infinite loop:
function module() {
'use asm';
function f() {
var x = 42;
while ((x|0) > 32) {
}
}
return f;
};
module()();
| Assignee | ||
Comment 1•11 years ago
|
||
This is a work-around, I will try to come with a cleaner patch, as we
probably don't want to flsuh all the time.
| Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8614124 -
Flags: review?(benj)
| Assignee | ||
Updated•11 years ago
|
Attachment #8614123 -
Attachment is obsolete: true
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → nicolas.b.pierron
| Reporter | ||
Comment 3•11 years ago
|
||
Comment on attachment 8614124 [details] [diff] [review]
JitSpewer: Always flush after dumping the graph to the files.
Review of attachment 8614124 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks! It fixed the issue I saw when debugging a rather large asm.js module.
Attachment #8614124 -
Flags: review?(benj) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•