Closed Bug 415568 Opened 17 years ago Closed 13 years ago

Use new NSPR atomic macros - JS

Categories

(Core :: JavaScript Engine, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 592557

People

(Reporter: swsnyder, Assigned: swsnyder)

References

Details

(Keywords: perf)

Attachments

(1 file, 1 obsolete file)

Use the recently-added NSPR atomic Increment/Decrement/Set/Add macros rather than function calls in existing code. The goal is to improve general performance by moving those atomic operations to inline code. The inline code is much faster than the function calls at the cost of increasing the code size by about 8 bytes per use. For platforms that do not support the use of inline atomic operations, the original function calls are used, making this patch a no-op.
Marked as dependant on bug 415563 because that patch adds MSVC-compatible casting of parameters to atomic intrinsic functions.
Attached patch Reformatted, with CVS (deleted) — Splinter Review
Attachment #301309 - Attachment is obsolete: true
This was fixed by bug 592557. So dup of it.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: