RFR: 8352587: C2 SuperWord: we must avoid Multiversioning for PeelMainPost loops [v2]

[Emanuel Peter]

On Tue, 25 Mar 2025 07:30:42 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/loopTransform.cpp line 3435:
>> 
>>> 3433:       if (!peel_only) {
>>> 3434:         // We are going to add pre-loop and post-loop (PreMainPost).
>>> 3435:         // But should we also multi-version for auto-vectorization speculative
>> 
>> Just a side note: It seems that we sometimes say "multiversion" and sometimes "multi-version". I find the latter more readable. Maybe we can make it consistent at some point in a separate task. This could also mean that we have `*_multi_version_*` in method names instead of `*_multiversion*_`.
>
> Hmm... good catch. Personally, I prefer "multiversion" as a single word. But this is very subjective 😅

I now renamed the 2 occurances of `multi-version` to `multiversion`, since that was a minimal diff. We could still file an RFE to rename everything later.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24183#discussion_r2011493707