RFR: 8367657: C2 SuperWord: NormalMapping demo from JVMLS 2025 [v3]

Tue Sep 16 07:45:51 UTC 2025

On Tue, 16 Sep 2025 06:51:59 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix inlining
>
> test/hotspot/jtreg/compiler/gallery/NormalMapping.java line 149:
> 
>> 147:     }
>> 148: 
>> 149:     public static class Light {
> 
> Maybe add a quick comment what this class does since it's a demo and one might want to better understand what's going on. Same for `State` class below.

good idea!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27282#discussion_r2351272790