On Wed, 1 Jul 2026 00:09:36 GMT, Leonid Mesnik <[email protected]> wrote:

> The only general comment is that implementation doesn't check the 
> '--enable-preview' so agent with enabled 'can_support_value_objects' is not 
> going to fail. Do we need a separate test for this?

Thank you for looking at this change! This does not look as a real problem.
The runtime event paths already make the relevant behavioral decision from the 
actually allocated object:
  const bool is_inline = object->is_inline();

> Also, there is a general lack of coverage of this behaviour for multi 
> environments/agents.
> Might be needed to to file RFE to review 
> './vmTestbase/nsk/jvmti/scenarios/multienv' and see if they should be updated 
> or write new tests.

Yes. We have a lack of coverage in this area. I hope, it is not a stopper for 
this PR. :)

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

PR Comment: https://git.openjdk.org/valhalla/pull/2607#issuecomment-4850262184

Reply via email to