Request for review: 8015385 Remove RelaxAccessControlCheck for JDK 8 bytecodes

[David Holmes]

Hi Harold,

I'm not sure what the actual rules for this should be in terms of the 
accessor and accessee having different versions. But isn't the main 
issue the version of the accessee?

Further your change will now require RelaxAccessControlCheck  be true 
for pre 49 class files, but I thought this old code was supposed to 
continue working?

Thanks,
David

On 29/05/2013 11:25 PM, harold seigel wrote:
> Hi,
>
> Please review this fix for bug 8015385.
>
> This fix removes the use of the RelaxAccessControlCheck option for
> bytecode versions >= 52 and so requires the stricter access checking
> that previously could be avoided by using this option. Note that the
> original purpose of this option was for use with pre-Java_1_5 bytecodes
> that did not contain the necessary accessors needed to properly access
> private fields.
>
> This change was tested with JCK lang and vm tests, UTE vm.quick.testlist
> and vm.mlvm.testlist tests, JPRT, and jtreg tests. Tests were run on
> Linux and Solaris.
>
> Open webrev at http://cr.openjdk.java.net/~hseigel/bug_8015385/
> <http://cr.openjdk.java.net/%7Ehseigel/bug_8015385/>
>
> Bug link at http://bugs.sun.com/view_bug.do?bug_id=8015385
>
> Thanks, Harold