On Wed, 26 Jan 2022 08:36:44 GMT, Thomas Stuefe <stu...@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

Good to me.

-------------

Marked as reviewed by zgu (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7213

Reply via email to