RFR (tedious) 8173070: Remove ValueObj class for allocation subclassing for runtime code

[Thomas Stüfe]

Hi Coleen,

On Thu, Mar 8, 2018 at 7:26 PM, <coleen.phillimore at oracle.com> wrote:

>
> This change removes VALUE_OBJ_CLASS_SPEC as subclass for classes in the
> runtime code.  I decided to split this into 3 parts to divide the
> clicking.  See the bug for discussion of why we would like to remove this
> null class on most platforms in favor of the link time check to disallow
> Hotspot code from calling global operators new and delete (bug
> https://bugs.openjdk.java.net/browse/JDK-8198243)
>
> Tested with mach5 nightly tests with the full set of changes and mach5
> tier1-2 with this set.
>
> open webrev at http://cr.openjdk.java.net/~coleenp/8173070.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8173070
>
> I'll update the copyrights with hg commit.
>
> Thanks,
> Coleen
>

Looks mostly okay. Thank you for doing this, this will make C++ parsing in
CDT easier.

allocation.hpp: the comment around _ValueObj now reads strange. Also, just
curious, do we still need _ValueObj?

Kind Regards, Thomas