Closed Bug 590275 Opened 14 years ago Closed 14 years ago

JM: Clean up JSScript hacks

Categories

(Core :: JavaScript Engine, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dvander, Assigned: dvander)

References

Details

Attachments

(1 file)

Attached patch fixSplinter Review
fix
Attachment #468772 - Flags: review?(dmandelin)
This patch takes all the random packing hacks in the compiler and separates them into a struct. The only remaining hack is that one buffer is allocated, like JSScript.
Comment on attachment 468772 [details] [diff] [review]
fix

I hope we continue improving this as we get the chance. This is definitely a step in the right direction.
Attachment #468772 - Flags: review?(dmandelin) → review+
Pushed with some follow-up fixes to places still referencing nmap[-1].

http://hg.mozilla.org/projects/jaegermonkey/rev/35fed487745d
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Depends on: 590749
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: