RFR: JDK-8318671: Potential uninitialized uintx value after JDK-8317683 [v3]
Thomas Stuefe
stuefe at openjdk.org
Thu Oct 26 09:02:12 UTC 2023
On Thu, 26 Oct 2023 08:58:13 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> When using 'MemStat' CompileCommand, we accidentally register the command if an invalid suboption had been specified. Fixed, added regression test (verified).
>
> Thomas Stuefe has updated the pull request incrementally with two additional commits since the last revision:
>
> - remove tab
> - Make patch more palatable
test/hotspot/jtreg/serviceability/dcmd/compiler/CompilerMemoryStatisticTest.java line 53:
> 51: * @run main/othervm -XX:CompileCommand=memstat,*.*,collect CompilerMemoryStatisticTest
> 52: */
> 53:
Reviewer hint: Removed because its redundant with the syntax check introduced in MemStatTest.java
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16335#discussion_r1372819967
More information about the hotspot-compiler-dev
mailing list