RFR: JDK-8280583: Always build NMT [v3]
Aleksey Shipilev
shade at openjdk.java.net
Wed Jan 26 08:45:33 UTC 2022
On Wed, 26 Jan 2022 08:36:44 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> After discussing this on hotspot-runtime-dev [1], the general opinion seems to be that it would be worthwhile to get rid of INCLUDE_NMT and make NMT unconditional. This affects minimal builds only. As pointed out in the mail thread, the overhead is very small and it would get rid of one configuration to build and test.
>>
>> [1] https://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2022-January/053504.html
>>
>> Patch removes INCLUDE_NMT from hotspot, as well as dependent macros, as well as the associated build option.
>
> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>
> - fix copyrights
> - remove INCLUDE_NMT and dependend code
This looks fine to me.
-------------
Marked as reviewed by shade (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/7213
More information about the build-dev
mailing list