RFR: 8325589: C2 SuperWord refactoring: create VLoopAnalyzer with Submodules [v3]

Emanuel Peter epeter at openjdk.org
Thu Feb 15 23:15:58 UTC 2024


On Wed, 14 Feb 2024 14:38:25 GMT, Christian Hagedorn <chagedorn 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 eight commits:
>> 
>>  - manual merge
>>  - move _loop_or_ctrl from ResouceArena, bc ResourceMark in SuperWord::dependence_graph
>>  - remove some comments
>>  - VLoopTypes
>>  - VLoopBody
>>  - VLoopMemorySlices
>>  - VLoopReductions
>>  - 8325589
>
> Nice refactoring into submodules! Here are some first comments. I have to do another iteration looking closer at the interplay of all the new submodules.

@chhagedorn Ok, I changed back to `_vloop` fields.
We can discuss if I should revert `igvn, phase, cl, ...` to fields as well. But that change was in a different changeset, and so I would also do that change in a future RFE.

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

PR Comment: https://git.openjdk.org/jdk/pull/17800#issuecomment-1947491632


More information about the hotspot-compiler-dev mailing list