On Wed, 19 Apr 2023 02:22:38 GMT, David Holmes <dhol...@openjdk.org> wrote:
> @mgronlun I agree this does not look good. I'm not sure this was the right > way to conditionalize the new code, rather than ensuring the callsites were > conditionalized on INCLUDE_JVMTI. It follows the same pattern as for other jvmti*.cpp files also excluded via make/hotspot/lib/jvmFeatures.gmk. For example, jvmtiExport.cpp. Might be better to improve with conditionalized callsites, agree. ------------- PR Comment: https://git.openjdk.org/jdk/pull/13512#issuecomment-1514547771
