RFR (S): 8133470: Uses of Atomic methods in plab.hpp should be moved to .inline.hpp file

Thomas Schatzl thomas.schatzl at oracle.com
Thu Aug 13 10:49:39 UTC 2015


Hi all,

  in the review for 8073013 one of the review questions was why plab.hpp
added atomic.inline.hpp, and that that is not good practice.

This has been a pre-existing bug, i.e. plab.hpp already used the Atomic
functions that reside in .inline.hpp.

This CR fixes this issue by moving the problematic calls into
plab.inline.hpp.

The only problem is that changing includes from plab.hpp to
plab.inline.hpp in cpp files and adding plab.hpp in header files opens
up dependency issues and other missing includes.

That's why this change is larger than it should.

Webrev:
https://bugs.openjdk.java.net/browse/JDK-8133470
CR:
http://cr.openjdk.java.net/~tschatzl/8133470/webrev/

Testing:
jprt

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list