Last Comment Bug 758613 - Condense JIT handle information in scripts
: Condense JIT handle information in scripts
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Other Branch
: x86 Mac OS X
-- normal (vote)
: mozilla15
Assigned To: Brian Hackett (:bhackett)
: Jason Orendorff [:jorendorff]
Depends on: 759719 761396
Blocks: 750834
  Show dependency treegraph
Reported: 2012-05-25 07:23 PDT by Brian Hackett (:bhackett)
Modified: 2012-09-23 11:06 PDT (History)
3 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (670ebafda11a) (15.96 KB, patch)
2012-05-25 07:23 PDT, Brian Hackett (:bhackett)
dvander: review+
Details | Diff | Splinter Review

Description User image Brian Hackett (:bhackett) 2012-05-25 07:23:09 PDT
Created attachment 627222 [details] [diff] [review]
patch (670ebafda11a)

After bug 750834 scripts can now have up to four JITScripts simultaneously (normal/ctor versions with or without barriers), bloating the number of handles stored in the JSScript itself.  Usually, all these fields are NULL, so it would be better if the handles were stored in a separate structure which is non-NULL if the script has had any JIT activity (purging the structure whenever all JIT code for the script is released).
Comment 1 User image Brian Hackett (:bhackett) 2012-05-25 08:21:03 PDT
Comment 2 User image Ryan VanderMeulen [:RyanVM] 2012-05-26 05:13:04 PDT

Note You need to log in before you can comment on or make changes to this bug.