RFR(S): 8186439: [MVT] ClassFileParser should ignore JVM_ACC_VALUE for class file version < 53.1

[David Simms]

src/share/vm/oops/instanceKlass.hpp: whitespace changes, from 2 to 4 
spaces (style guide says 2) ? mistake, forgot to revert ?

Otherwise, looks good.

Minor point, "supports_value_types()" and with "EnableValhalla || 
EnableMVT" ?

/David Simms

On 21/08/17 10:22, Tobias Hartmann wrote:
> Hi,
>
> when debugging Roland's fix for 8185265, I noticed that some JCK tests 
> failed because we were treating a klass with JVM_ACC_NATIVE (0x100) as 
> value klass. The JVM spec says:
> "All bits of the access_flags item not assigned in Table 4.1-A are 
> reserved for future use. They should be set to zero in generated class 
> files and should be ignored by Java Virtual Machine implementations." [1]
>
> Therefore, the JVM_ACC_VALUE (== JVM_ACC_NATIVE) flag should be 
> ignored for class file versions < 53.1:
> http://cr.openjdk.java.net/~thartmann/8186439/webrev.00/
>
> Thanks,
> Tobias
>
> [1] https://docs.oracle.com/javase/specs/jvms/se8/html/jvms-4.html