JSContext.operationCount is one of the most frequently used fields in JSContext code-wise and, at least for scripts with tight loops, the most frequently accessed fields of JSContext runtime-wise. Thus we should consider making it at the first field in JSContext. See bug 450000 comment 61 - 64.
Created attachment 348554 [details] [diff] [review] fix v1 The patch moves the operationCount field to the first position in JSContext and adjust JSCList-to-JSContext conversion code accordingly.
Created attachment 348555 [details] [diff] [review] fix v2 The new version name JSCList-to-JSContext conversion function as js_ContextFromLinkField. The former name js_ThreadLinkToContext is wrong as JSContext.link has nothing to do with threads.
Attachment #348555 - Flags: approval1.9.1b2? → approval1.9.1?
Comment on attachment 348555 [details] [diff] [review] fix v2 Spoke with brendan, should be ok to take this post-b2
Comment on attachment 348555 [details] [diff] [review] fix v2 a191=beltzner
Attachment #348555 - Flags: approval1.9.1? → approval1.9.1+
merged to mc
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.