RFR: 8312596: Null pointer access in Compile::TracePhase::~TracePhase after JDK-8311976

[Ashutosh Mehra]

On Thu, 27 Jul 2023 06:15:01 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Please review this PR to fix a potential null pointer access in using `_compile`.
>> Updated the code to unconditionally initialize `_compile` and added an assert (similar to C1's `PhaseTraceTime` constructor) for it to be non-null.
>
> You can put it in `compiler/c2` like the following test, which is also a sanity test for a C2 flag:
> https://github.com/openjdk/jdk/blob/master/test/hotspot/jtreg/compiler/c2/TestVerifyGraphEdges.java
> (You can drop `-Xbatch` which is implied by `-Xcomp`)

@chhagedorn

> You can put it in compiler/c2 like the following test, which is also a sanity test for a C2 flag:

Thanks for the suggestion. I have added the sanity test. Please review.

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

PR Comment: https://git.openjdk.org/jdk/pull/15002#issuecomment-1653638242