Code Review for 6996240

Prepared by:twisti on Tue Nov 2 10:53:11 PDT 2010
Workspace:/home/twisti/hotspot-comp/6996240
Compare against: /home/twisti/hotspot-comp/hotspot
Summary of changes: 31 lines changed: 1 ins; 0 del; 30 mod; 9877 unchg
Patch of changes: 6996240.patch
Printable review: 6996240.pdf
Author comments:
6996240: The BitSet.length method sometimes returns an index+1 value less than that of the highest bit set.
Reviewed-by:

It's a problem with countLeadingZerosL. dst is a iRegI but is used as
temporary for an iRegL OR value.

The fix is to use a second temporary iRegL register for the OR value.

Tested with the failing testcase attached to the CR.
Bug id: 6996240 The BitSet.length method sometimes returns an index+1 value less than that of the highest bit set.
Legend: Modified file
Deleted file
New file

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

31 lines changed: 1 ins; 0 del; 30 mod; 9877 unchg

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