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

[Vladimir Kozlov]

On Wed, 5 Nov 2025 09:17:44 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/vectorization.cpp line 580:
>> 
>>> 578: 
>>> 579:   float sum = 0;
>>> 580:   for (int j = 0; j < body().body().length(); j++) {
>> 
>> What is `body().body()` mean?
>
> `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!

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

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