RFR: 8324890: C2 SuperWord: refactor out VLoop, make unrolling_analysis static, remove init/reset mechanism [v2]

Vladimir Kozlov kvn at openjdk.org
Fri Feb 2 18:32:05 UTC 2024


On Fri, 2 Feb 2024 09:01:08 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/loopnode.cpp line 4866:
>> 
>>> 4864: 
>>> 4865:   // Auto-vectorize main-loop
>>> 4866:   if (C->do_superword() && C->has_loops() && !C->major_progress()) {
>> 
>> Should we rename `do_superword()` too?
>
> @vnkozlov yes, but later, I would say. It simply caries the same values as `UseSuperWord`. I guess eventually we may have different AutoVectorization components, and it may be better to have a unified AutoVectorization option. Is it ok to leave that to the future, and leave it in for now?

Yes, we can do it for later.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17624#discussion_r1476514848


More information about the hotspot-compiler-dev mailing list