Closed Bug 415568 Opened 16 years ago Closed 12 years ago

Use new NSPR atomic macros - JS

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 592557

People

(Reporter: swsnyder, Assigned: swsnyder)

References

Details

(Keywords: perf)

Attachments

(1 file, 1 obsolete file)

Reformatted, with CVS
3.54 KB, patch
Details | Diff | Splinter Review 
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.


        Attachment #301309 -
        Attachment is obsolete: true

    
    

    
  
This was fixed by bug 592557.  So dup of it.
Status: NEW → RESOLVED
Closed: 12 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: