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

Stefan Karlsson stefan.karlsson at oracle.com
Tue Aug 16 11:22:13 UTC 2016


Looks good!

Thanks,
StefanK

On 2016-08-16 03:01, 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.
>
> Thanks,
> David



More information about the hotspot-dev mailing list