On Tue, 31 Mar 2026 09:18:25 GMT, Maurizio Cimadamore <[email protected]> wrote:
>> I agree with excluding `Linker` from this PR - I've felt `Linker` does not >> need to be value-based TBH. In addition `FunctionDescriptor` does not need >> to be value based because we would not use `==` for it. >> >> So I think it should do first to exclude `Linker` and `FunctionDescriptor` >> from value based class (I think it needs CSR). Then we can go ahead this PR. >> What do you think? > >> I agree with excluding `Linker` from this PR - I've felt `Linker` does not >> need to be value-based TBH. In addition `FunctionDescriptor` does not need >> to be value based because we would not use `==` for it. >> >> So I think it should do first to exclude `Linker` and `FunctionDescriptor` >> from value based class (I think it needs CSR). Then we can go ahead this PR. >> What do you think? > > In my mind, FunctionDescriptor and MemoryLayout are very close, as the former > is just an aggregate of the latter. For instance, the Linker uses FD as keys > in maps -- and I suspect others might want to do so as well. That said, I > agree that, given the fact that FD contains collections/arrays, we can't > really recommend people to use `==` for it. But still, it feels like a value > class to me (in spirit). @mcimadamore As I mentioned, I replied to your comment on https://github.com/openjdk/jdk/pull/30481#issuecomment-4196887420 . Could you check it? I want to work #30481 before this PR. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30443#issuecomment-4211400060
