RFR: 8340093: C2 SuperWord: implement cost model [v3]

Emanuel Peter epeter at openjdk.org
Thu Nov 6 08:16:06 UTC 2025


On Wed, 5 Nov 2025 15:53:15 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> `VLoopAnalyzer` (`this`) has multiple analysis subcomponents. One of them is the `VLoopBody`, i.e. `this->body()` / `this->_body.` And it has access to a `GrowableArray` `body()`, which maps the nodes of the loop.
>> 
>> Maybe `loopBody().nodes()` would sound better here. If you prefer that, I file a separate renaming RFE.
>
> Yes, would be nice if you move `body().body()` into separate method with comment explaining it. Thanks!

FYI, I filed: [JDK-8371391](https://bugs.openjdk.org/browse/JDK-8371391) C2 SuperWord: rename body().body() to something more understandable

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27803#discussion_r2497934619


More information about the hotspot-compiler-dev mailing list