Code Review for hotspot

Prepared by:Andreas Schoesser on Fri Jul 18 09:05:53 CEST 2014
Workspace:/net/usr.work/d051827/OpenJDK/hs-comp/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version:6727
Summary of changes: 11 lines changed: 11 ins; 0 del; 0 mod; 392 unchg
Changeset: hotspot.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/ci/ciField.cpp

rev 6728 : 8050978: Fix bad field access check in C1 and C2

Summary: JCK8 test vm/constantpool/accessControl/accessControl004/accessControl00402m3/accessControl00402m3.html fails with -Xbatch -Xcomp due to bad field access check in C1 and C2. Fix: In ciField::ciField(), just before the canonical holder is stored into the _holder variable (and which is used by ciField::will_link()) perform an additional access check with the holder declared in the class file. If this check fails, store the declared holder instead and ciField::will_link() will bail out compilation for this field later on. Then, the interpreter will throw an PrivilegedAccessException at runtime.

Contributed-by: andreas.schoesser@sap.com
11 lines changed: 11 ins; 0 del; 0 mod; 392 unchg

This code review page was prepared using /sapmnt/home1/d051827/myscripts/webrev.ksh (vers 25.5-hg+openjdk.java.net).