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

Severin Gehwolf sgehwolf at redhat.com
Wed Aug 17 08:23:05 UTC 2016


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.

Cheers,
Severin



More information about the hotspot-dev mailing list