Closed Bug 413498 Opened 17 years ago Closed 17 years ago

Rudimentary support for "finally" clauses

Categories

(Tamarin Graveyard :: Self-hosting compiler (ESC), defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: lhansen, Unassigned)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
Supports basic "finally" handling.  Does not support break, continue, or return past a finally clause -- the verifier has real problems with these (even in ASC according to Erik).  A syntax error is thrown if the program tries to do this.
Attachment #298473 - Flags: review?(tierney)
Comment on attachment 298473 [details] [diff] [review]
Patch

Looks good.  ASC does go to great lengths to satisfy the verifier, but still fails on some edge cases.  However, it does produce working code for the majority of cases.  I'm not sure if it's impossible to get it right, or just very, very difficult.  

We may have to do something similar for ESC, but we can worry about that later.
Attachment #298473 - Flags: review?(tierney) → review+
Pushed as changeset 328.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: