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

Thomas Stüfe thomas.stuefe at gmail.com
Thu Mar 8 20:31:58 UTC 2018

On Thu, Mar 8, 2018 at 9:30 PM, Thomas Stüfe <thomas.stuefe at gmail.com> wrote:

Hi Coleen,

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

This change removes VALUEOBJCLASSSPEC 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? Oh, just read that others already commented on the comment. So, never mind...

Kind Regards, Thomas

More information about the hotspot-dev mailing list