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