Code Review for 6823354

Prepared by:twisti on Mon May 4 21:36:33 CEST 2009
Workspace:/export/home/twisti/hotspot-comp/6823354
Compare against: /export/home/twisti/hotspot-comp/hotspot
Compare against version:742
Summary of changes: 1000 lines changed: 984 ins; 8 del; 8 mod; 59873 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}.{numberOfLeadingZeros,numberOfTrailingZeros}()
Legend: Modified file
Deleted file
New file

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

rev 743 : [mq]: 6823354
156 lines changed: 156 ins; 0 del; 0 mod; 9381 unchg

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

rev 743 : [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 743 : [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 743 : [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 743 : [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 743 : [mq]: 6823354
154 lines changed: 154 ins; 0 del; 0 mod; 13788 unchg

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

rev 743 : [mq]: 6823354
122 lines changed: 122 ins; 0 del; 0 mod; 12967 unchg

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

rev 743 : [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 743 : [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.cpp 

rev 743 : [mq]: 6823354
82 lines changed: 82 ins; 0 del; 0 mod; 1253 unchg

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

rev 743 : [mq]: 6823354
52 lines changed: 44 ins; 4 del; 4 mod; 649 unchg

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

rev 743 : [mq]: 6823354
52 lines changed: 52 ins; 0 del; 0 mod; 5133 unchg

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

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

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

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

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

rev 743 : [mq]: 6823354
266 lines changed: 266 ins; 0 del; 0 mod; 0 unchg

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