RFR: 8343685: C2 SuperWord: refactor VPointer with MemPointer [v4]

Emanuel Peter epeter at openjdk.org
Tue Jan 14 07:34:50 UTC 2025


On Mon, 13 Jan 2025 19:50:24 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 116 commits:
>> 
>>  - copyright 2025
>>  - Merge branch 'master' into JDK-8343685-VPointer-MemPointer
>>  - manual merge
>>  - fix printing
>>  - rename
>>  - fix up print
>>  - add TestEquivalentInvariants.java
>>  - improve documentation
>>  - hide parser via delegation
>>  - Merge branch 'master' into JDK-8343685-VPointer-MemPointer
>>  - ... and 106 more: https://git.openjdk.org/jdk/compare/84e6432b...b64f9295
>
> src/hotspot/share/opto/mempointer.cpp line 243:
> 
>> 241:     // is too deep. The constant is chosen arbitrarily, not too large but big
>> 242:     // enough for all normal cases.
>> 243:     if (worklist.length() > 100) { return false; }
> 
> May be specify size when creating `worklist` so there is no need for resizing when it is grow.

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21926#discussion_r1914364605


More information about the hotspot-compiler-dev mailing list