RFR (S): 8193063: Enabling narrowOop values for RawAccess accesses

[Erik Österlund]

Hi Per,

Thank you for the review.

/Erik

On 2018-01-09 11:18, Per Liden wrote:
> Looks good!
>
> /Per
>
> On 2017-12-05 15:49, Erik Österlund wrote:
>> Hi,
>>
>> In order to replace oopDesc::load_heap_oop() but with stricter memory
>> ordering properties, like MO_VOLATILE, some Access tweaks are required
>> to allow RawAccess<>::oop_load() to return narrowOop values.
>>
>> I made the necessary changes to allow narrowOop values for all RawAccess
>> operations that have an address (not the _at variants).
>>
>> While I am at it, I thought I'd clean up a few things that bug me:
>>
>> * The decorator verification for memory ordering specifically did not
>> work as I originally intended, leading to harder to decipher compiler
>> errors deeper down when using the wrong memory ordering decorators
>> * An unnecessary include of oop.inline.hpp was removed from the G1
>> barrier set.
>>
>> This change helps solving the following bug:
>> https://bugs.openjdk.java.net/browse/JDK-8129440
>>
>> This bug is filed under:
>> https://bugs.openjdk.java.net/browse/JDK-8193063
>>
>> Webrev:
>> http://cr.openjdk.java.net/~eosterlund/8193063/webrev.00/
>>
>> Thanks,
>> /Erik