Code Review for 7090654

Prepared by:never on Wed Sep 14 12:32:04 PDT 2011
Workspace:/net/smite.us.oracle.com/export/ws/synth
Compare against: ssh://hg.openjdk.java.net/hsx/hotspot-comp-gate/hotspot
Summary of changes: 21 lines changed: 0 ins; 6 del; 15 mod; 2496 unchg
Patch of changes: 7090654.patch
Author comments:
7090654: nightly failures after 7086585
Reviewed-by:

I had made late changes to accessing of fields in InstanceKlass and
didn't rerun my tests so I create a mismatch between the indexing.
Some code was using i * 7 and others was using i. I've corrected it
so it's using i consistently everything. Tested with sajdi.

I screwed up the new of fields when we recreate a .class file from our
internal representation. It should be unscaled. Tested with
java/lang/instrument regression tests. I had thought the tests I
originally ran exercised this path but I was wrong.

Bug id: Bug Database
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw agent/src/share/classes/sun/jvm/hotspot/oops/InstanceKlass.java

19 lines changed: 0 ins; 6 del; 13 mod; 924 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw agent/src/share/classes/sun/jvm/hotspot/runtime/VM.java

1 line changed: 0 ins; 0 del; 1 mod; 875 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiClassFileReconstituter.cpp

1 line changed: 0 ins; 0 del; 1 mod; 697 unchg

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