On Thu, 18 Jun 2026 17:14:10 GMT, Alan Bateman <[email protected]> wrote:

>> I think Roger maybe added this constructor out of convenience. I checked 
>> some VM code and they seem to be never using this constructor; they have 
>> various ways of constructing messages, often incorporating the action that 
>> requires identity. Maybe we can consider removing this constructor too.
>
> That might be the simplest, it would always be added later if there is a 
> needed.

That makes sense. I've now updated the PR to remove that constructor and since 
we are now removing this constructor, I looked for references of the other 2 
which take a `Throwable`. Those aren't used either. So I've removed them as 
well.

I've run tier1, tier2 and tier3 with this change and it completed without any 
related issues.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2562#discussion_r3439817986

Reply via email to