RFR: 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp

[David Holmes]

On 17/08/2016 6:23 PM, Severin Gehwolf wrote:
> On Tue, 2016-08-16 at 14:22 +0200, Severin Gehwolf wrote:
>> On Tue, 2016-08-16 at 17:01 +1000, David Holmes wrote:
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8157907
>>>
>>> webrev: http://cr.openjdk.java.net/~dholmes/8157907/webrev/
>>>
>>> Following on from the discussion here:
>>>
>>> http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-August/024171.html
>>>
>>> this is the formal RFR for these changes.
>>>
>>> Summary:
>>> - fold atomic.inline.hpp into atomic.hpp
>>> - rename atomic_*.inline.hpp files to atomic_*.hpp
>>> - only include atomic.hpp where needed
>>> - remove unnecessary additional includes
>>>
>>> I need someone to confirm these changes build fine on: ppc, aarch64 and
>>> zero please.
>>
>> I'll test this with Zero. Should have results later today.
>
> FYI: Zero works fine with this.

Great! Thanks for testing that for me Severin!

David

> Cheers,
> Severin
>