Closed Bug 684037 Opened 13 years ago Closed 13 years ago

IonMonkey: Assert building SSA finding target of break in while

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dvander, Assigned: dvander)

References

Details

Attachments

(1 file)

This is failing on ecma_2/Statements/while-004.js.
Attached patch fixSplinter Review
Attachment #557718 - Flags: review?(sstangl)
Note, this patch just changes the assert to a compilation-abort. This case seems extremely rare, and we'd need to add more information to the bytecode to parse it in our current algorithm.
OS: Linux → All
Hardware: x86_64 → All
Comment on attachment 557718 [details] [diff] [review]
fix

Review of attachment 557718 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/ion/IonBuilder.cpp
@@ +1009,5 @@
> +        // single-level break is actually 5. To recognize this case we'd need
> +        // to know about the label structure at 0,5 ahead of time - and lacking
> +        // those source notes for now, we just abort instead.
> +        abort("could not find the target of a break");
> +        return ControlStatus_Error;

This is fine, but we should plan to handle the case eventually. Can a different bug be filed for this, with its number referenced in a comment here?
Attachment #557718 - Flags: review?(sstangl) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/834f8ff1748e

Filed bug 684384.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.