Code Review for 7088020

Prepared by:never on Fri Sep 9 11:34:54 PDT 2011
Workspace:/net/smite.us.oracle.com/export/ws/align
Compare against: /net/smite.us.oracle.com/export/ws/synth
Summary of changes: 153 lines changed: 87 ins; 50 del; 16 mod; 12570 unchg
Patch of changes: 7088020.patch
Author comments:
7088020: SEGV in JNIHandleBlock::release_block
Reviewed-by:

The throw_WrongMethodTypeException stub on x64 needs to align the
stack before calling into the runtime or it might crash. I also
noticed that two stubs were dead which made an extra argument dead so
I cleaned that up at the same time. Tested on linux-amd64 with new
regression test and failing tests from report.

Bug id: 7088020 SEGV in JNIHandleBlock::release_block
Legend: Modified file
Deleted file
New file

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

12 lines changed: 0 ins; 7 del; 5 mod; 3498 unchg

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

5 lines changed: 5 ins; 0 del; 0 mod; 2431 unchg

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

12 lines changed: 0 ins; 6 del; 6 mod; 2356 unchg

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

32 lines changed: 2 ins; 25 del; 5 mod; 3124 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/zero/vm/stubGenerator_zero.cpp

6 lines changed: 0 ins; 6 del; 0 mod; 268 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/stubRoutines.cpp

2 lines changed: 0 ins; 2 del; 0 mod; 509 unchg

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

4 lines changed: 0 ins; 4 del; 0 mod; 384 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/7088020/Test7088020.java

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

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