RFR: 8326962: C2 SuperWord: cache VPointer [v6]
Emanuel Peter
epeter at openjdk.org
Wed Apr 3 07:17:34 UTC 2024
On Wed, 3 Apr 2024 06:38:45 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> With the new `for_each_mem`, the code is already much easier to read. I don't know if splitting it further would really help now?
>
> That's already better!
>
> My general take on that is when I see:
>
> // Do x
> // Do y
> // Do z
>
> it suggests that it should actually be
>
> x();
> y();
> z();
>
> But that's just my personal preference :-)
Ok, I split it up now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18577#discussion_r1549054681
More information about the hotspot-compiler-dev
mailing list