Integrated: 8325589: C2 SuperWord refactoring: create VLoopAnalyzer with Submodules

[Emanuel Peter]

On Sat, 10 Feb 2024 17:52:46 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> Subtask of https://github.com/openjdk/jdk/pull/16620
> 
> **Goals**:
> - **modularity**: SuperWord is a nasty large class, and it is difficult to know what impacts what.
> - **code reuse**: In the future, we may want to reuse some components of SuperWord in other Vectorizers. For example the post-loop-vectorizer ([JDK-8308994](https://bugs.openjdk.org/browse/JDK-8308994)).
> 
> Hence, I pull some parts out of SuperWord, and put them in a new class `VLoopAnalyzer`, with submodules:
> 
> VLoopAnalyzer
>   VLoopReductions
>   VLoopMemorySlices
>   VLoopBody
>   VLoopTypes
> 
> 
> **Future Work**
> In my draft for https://github.com/openjdk/jdk/pull/16620, I also created a submodule `VLoopDependenceGraph`.
> But that is also a lot of code change.
> I decided to do this in a separate RFE, to keep this patch here reasonably short.
> 
> I also left many functions in `superword.cpp`, even though their classes are in `vectorization.hpp`.
> I think it is easier to review them in their old palces, with minor changes, for now.
> In a future RFE, I can then cleanly copy them, without any changes to them.

This pull request has now been integrated.

Changeset: 490825fb
Author:    Emanuel Peter <epeter at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/490825fb915c9a63459f87ffea4a9b7c1f0448ef
Stats:     894 lines in 7 files changed: 453 ins; 209 del; 232 mod

8325589: C2 SuperWord refactoring: create VLoopAnalyzer with Submodules

Reviewed-by: kvn, chagedorn

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

PR: https://git.openjdk.org/jdk/pull/17800