RFR: 8369187: Add wrapper for <new> that forbids use of global allocation and deallocation functions [v3]
Stefan Karlsson
stefank at openjdk.org
Mon Nov 17 09:13:43 UTC 2025
On Sun, 16 Nov 2025 01:10:40 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> 8369187: Add wrapper for <new> that forbids use of global allocation and deallocation functions
>>
>> Please review this change that adds `cppstdlib/new.hpp` as a wrapper for
>> including `<new>`. All existing inclusions of `<new>` are changed to include
>> the new wrapper.
>>
>> In additional to including `<new>`, this wrapper also provides deprecation
>> declarations to prevent the use of some facilities by HotSpot code.
>>
>> However, those deprecations need to be conditionalized to not apply to gtests,
>> so this change also adds a macro definition provided by the build system for
>> use in detecting that a header is being included by a gtest.
>>
>> Testing: mach5 tier1
>
> Kim Barrett has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>
> - Merge branch 'master' into wrap-stdlib-new
> - poison implicit alloc/dealloc in globalDefinitions
> - Merge branch 'master' into wrap-stdlib-new
> - further conditionalize deprecation of hardare interference sizes
> - add wrapper for <new>
The last change seems reasonable to me.
-------------
Marked as reviewed by stefank (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/28250#pullrequestreview-3471725618
More information about the build-dev
mailing list