Integrated: 8325064: C2 SuperWord: refactor construct_bb

Emanuel Peter epeter at openjdk.org
Mon Feb 5 16:31:13 UTC 2024


On Wed, 31 Jan 2024 16:53:59 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> Subtask of https://github.com/openjdk/jdk/pull/16620
> 
> The goal is to further disentangle different "components" in Superword.
> 
> In this refactoring, I disentangle the `bb`, `reduction` and `memory_slice` "components" which were all intertwined in `construct_bb`.
> 
> 1. Move memory slice code -> `analyze_memory_slices`.
> 2. Remove reduction checking code -> simply use the `is_marked_reduction_loop` condition outside.
> 3. `_data_entry`: was used for non-CFG nodes in the loop that have no input node that is also inside the loop. But that actually never happens! I removed that array, and replaced the code with verification.

This pull request has now been integrated.

Changeset: 89e6a02e
Author:    Emanuel Peter <epeter at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/89e6a02e3b095999199a4c1bac4deb6c9f31fe8e
Stats:     182 lines in 2 files changed: 70 ins; 84 del; 28 mod

8325064: C2 SuperWord: refactor construct_bb

Reviewed-by: kvn, chagedorn

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

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


More information about the hotspot-compiler-dev mailing list