Closed
Bug 617246
Opened 14 years ago
Closed 13 years ago
Cdecl refactoring
Categories
(Tamarin Graveyard :: Virtual Machine, enhancement)
Tamarin Graveyard
Virtual Machine
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alexmac, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
92.63 KB,
patch
|
stejohns
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_5; en-us) AppleWebKit/533.19.4 (KHTML, like Gecko) Version/5.0.3 Safari/533.19.4 Build Identifier: The diff looks gigantic but only because I fixed the whitespace and removed tabs throughout the file, view it ignoring ws to see the functional changes. Some of the work to enable exact callmethod for aot required small changes to the argument coercion code used in cdecl and fixed some bugs with the size of the argdesc computation. Reproducible: Always
Reporter | ||
Comment 1•14 years ago
|
||
Attachment #495754 -
Flags: review?(stejohns)
Comment 2•14 years ago
|
||
(In reply to comment #0) > The diff looks gigantic but only because I fixed the whitespace and removed > tabs throughout the file, view it ignoring ws to see the functional changes. For future reference, it's usually preferred to upload diffs without the whitespace fixes, and fix the whitespace when you push.
Updated•14 years ago
|
Attachment #495754 -
Flags: review?(stejohns) → review+
Reporter | ||
Comment 3•14 years ago
|
||
Attachment #495754 -
Attachment is obsolete: true
Updated•13 years ago
|
Attachment #499003 -
Flags: review+
Comment 4•13 years ago
|
||
TR 5851:bb1d051d4827
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•