RFR: 8370914: C2: Reimplement Type::join [v4]

Quan Anh Mai qamai at openjdk.org
Sat Nov 22 11:19:25 UTC 2025


On Sat, 22 Nov 2025 00:44:56 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - Merge branch 'master' into typejoin
>>  - Move dual to ASSERT only
>>  - Keep old version for verification
>>  - whitespace
>>  - Reimplement Type::join
>
> Also, I think it would make sense to target this for jdk 27 after the fork.

@dean-long I think you are misunderstanding, the answer is incorrect because the result it gives, `java/nio/file/spi/FileSystemProvider:AnyNull *,iid=top`, is empty, while the correct answer is the set which contains the single value `null`, and is not empty. The reason for this inaccuracy is that there are 2 LCAs for the inputs on the lattice, and they do not subtype each other, we choose the wrong one out of those LCAs.

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

PR Comment: https://git.openjdk.org/jdk/pull/28051#issuecomment-3566574122


More information about the hotspot-compiler-dev mailing list