Code Review for main-hotspot-g1_2

Prepared by:Martin Doerr, Lutz Schmidt on Wed Apr 24 10:35:06 CEST 2013
Workspace:/net/usr.work/d045726/oJ/contribs/8012715-g1_size_t_bug/main-hotspot-g1_2
Compare against: http://hg.openjdk.java.net/hsx/hotspot-main/hotspot
Compare against version:4494
Summary of changes: 28 lines changed: 4 ins; 12 del; 12 mod; 3829 unchg
Patch of changes: main-hotspot-g1_2.patch
Legend: Modified file
Deleted file
New file

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

rev 4495 : 8012715: Fix G1 bug in graphKit.cpp: Access of PtrQueue::_index was int not size_t.

In graphKit INT operations were generated to access PtrQueue::_index which
has type size_t.  This is 64 bit on 64-bit machines. No problems occur on
little endian machines as long as the index fits into 32 bit, but on
big endian machines the upper part is read, which is zero. This leads
to unnecessary branches to the slow path in the runtime.
28 lines changed: 4 ins; 12 del; 12 mod; 3829 unchg

This code review page was prepared using /usr/local/bin/webrev.ksh (vers 23.18-hg).