RFR: 8355230: Crash in fuzzer tests: assert(n != nullptr) failed: must not be null [v2]

[Roland Westrelin]

On Tue, 20 May 2025 10:06:24 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - test comment
>>  - test cleanup
>
> @rwestrel Thanks for the explanation!
> 
> I leave it up to you if you want to update the test with my suggestions, or not.

@eme64 @TobiHartmann I cleaned up the test a bit. I also added a comment.

> test/hotspot/jtreg/compiler/c2/TestNullRegionInputAtPhiMakePathDead.java line 28:
> 
>> 26:  * @bug 8355230
>> 27:  * @summary Crash in fuzzer tests: assert(n != nullptr) failed: must not be null
>> 28:  * @run main/othervm -XX:CompileCommand=compileonly,TestNullRegionInputAtPhiMakePathDead::* -Xcomp TestNullRegionInputAtPhiMakePathDead
> 
> What about a run without `Xcomp`?

It's unclear to me anything would get compiled then.

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

PR Comment: https://git.openjdk.org/jdk/pull/25268#issuecomment-2894140938
PR Review Comment: https://git.openjdk.org/jdk/pull/25268#discussion_r2097769351