RFR: 8312332: C2: Refactor SWPointer out from SuperWord [v3]
Pengfei Li
pli at openjdk.org
Wed Sep 13 10:00:07 UTC 2023
On Tue, 12 Sep 2023 16:51:31 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
> In general it looks good. I just want that you add more description.
>
> And I have question. Part of code is under "#ifndef PRODUCT". Did you build "optimized" VM to make sure it works?
@vnkozlov, I just checked with CompilerDirectives and got the same tracing output between "fastdebug" build and "optimized" build, and no tracing output is seen for "release" build.
> src/hotspot/share/opto/vectorization.hpp line 31:
>
>> 29: #include "opto/loopnode.hpp"
>> 30:
>> 31: // ----------------------------------VPointer----------------------------------
>
> We stop using such lines - there was cleaning RFE to remove them.
Fixed
> src/hotspot/share/opto/vectorization.hpp line 32:
>
>> 30:
>> 31: // ----------------------------------VPointer----------------------------------
>> 32: // Information about an address for dependence checking and vector alignment
>
> I am fine with using VPointer name but I think you need to add more description here to avoid confusion.
>
> May be add a general description what is in this file: "for shared logics and utilities for C2's auto-vectorization".
Done
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15013#issuecomment-1717317480
PR Review Comment: https://git.openjdk.org/jdk/pull/15013#discussion_r1324269957
PR Review Comment: https://git.openjdk.org/jdk/pull/15013#discussion_r1324269642
More information about the hotspot-compiler-dev
mailing list