Code Review for 6823354

Prepared by:twisti on Wed Apr 22 20:27:30 CEST 2009
Workspace:/export/home/twisti/hotspot-comp/6823354
Compare against: /export/home/twisti/hotspot-comp/hotspot
Compare against version:721
Summary of changes: 760 lines changed: 752 ins; 4 del; 4 mod; 58625 unchg
Patch of changes: 6823354.patch
Author comments:
The intrinsics use different approaches on the implemented
architectures. On x86 they use instructions like BSF, BSR, or LZCNT
if available. On SPARC they use the POPC instruction, but the
intrinsified versions are only used if the CPU has a hardware version
of POPC.

Extensive benchmark numbers can be found in the CR.
Bug id: 6823354 Add intrinsics for {Integer,Long}.{highestOneBit,numberOfLeadingZeros,numberOfTrailingZeros}()
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/sparc/vm/sparc.ad 

rev 722 : [mq]: 6823354
159 lines changed: 159 ins; 0 del; 0 mod; 9377 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/assembler_x86.cpp 

rev 722 : [mq]: 6823354
48 lines changed: 48 ins; 0 del; 0 mod; 8389 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/assembler_x86.hpp 

rev 722 : [mq]: 6823354
18 lines changed: 18 ins; 0 del; 0 mod; 2220 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/vm_version_x86.cpp 

rev 722 : [mq]: 6823354
9 lines changed: 8 ins; 0 del; 1 mod; 526 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/vm_version_x86.hpp 

rev 722 : [mq]: 6823354
19 lines changed: 13 ins; 4 del; 2 mod; 459 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/x86_32.ad 

rev 722 : [mq]: 6823354
157 lines changed: 157 ins; 0 del; 0 mod; 13788 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/x86_64.ad 

rev 722 : [mq]: 6823354
116 lines changed: 116 ins; 0 del; 0 mod; 12967 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/classfile/vmSymbols.hpp 

rev 722 : [mq]: 6823354
8 lines changed: 8 ins; 0 del; 0 mod; 967 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/classes.hpp 

rev 722 : [mq]: 6823354
4 lines changed: 4 ins; 0 del; 0 mod; 327 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/connode.hpp 

rev 722 : [mq]: 6823354
40 lines changed: 40 ins; 0 del; 0 mod; 657 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/library_call.cpp 

rev 722 : [mq]: 6823354
52 lines changed: 52 ins; 0 del; 0 mod; 5139 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/matcher.hpp 

rev 722 : [mq]: 6823354
7 lines changed: 6 ins; 0 del; 1 mod; 398 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/globals.hpp 

rev 722 : [mq]: 6823354
3 lines changed: 3 ins; 0 del; 0 mod; 3411 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/6823354/Test6823354.java

120 lines changed: 120 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /export/home/twisti/bin/webrev (vers 23.18-hg-never).