RFR: 8243208: Clean up JVMFlag implementation [v3]

Ioi Lam iklam at openjdk.java.net
Wed Sep 9 20:47:33 UTC 2020


On Wed, 9 Sep 2020 14:13:15 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed JVMFlag::is_writeable()
>
> src/hotspot/share/runtime/flags/jvmFlagLookup.hpp line 44:
> 
>> 42:     }
>> 43:     return h;
>> 44:   }
> 
> Can this function just call hash_code(s, strlen(s)) ?

strlen() is not constexpr so it can't be used here. I added my hand-rolled string_len function in
[bf90a37](https://github.com/openjdk/jdk/pull/82/commits/bf90a3765720267a8770f7bb0cc6957471ccb21b) so we can share a
single copy of the hash calculation loop.

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

PR: https://git.openjdk.java.net/jdk/pull/82


More information about the shenandoah-dev mailing list