RFR: 8325095: C2: bailout message broken: ResourceArea allocated string used after free [v2]

[Emanuel Peter]

On Sat, 17 Feb 2024 01:34:56 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Piling on... I opt for keeping this code simple. A list of subsequent failure states could easily be part of logging.
>
> Okay, I am voting for UL to print all failures we hit during compilation and keep reporting only one reason at the end.
> 
> log_error(compilation) (failiure_reason);
> 
> Or `log_warning()`

@vnkozlov @dean-long @tstuefe 
Ok, I mean logging of additional failure reasons might even be done in a separate RFE, right?
But don't we already log the failures? I mean there is the `log()->elem(` code.

And should I still keep the "most important" failure reason?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17710#discussion_r1494243314