RFR: 8349040: Test compiler/inlining/LateInlinePrinting.java fails after JDK-8319850

[Matthias Baesken]

On Thu, 30 Jan 2025 14:37:40 GMT, SendaoYan <syan at openjdk.org> wrote:

> > Hi, the unchanged test works with our debug builds so I am not sure if you need the added UnlockDiagnosticVMOptions switch; at least we did not need it in the nightly tests . Otherwise looks good to fix the issue in debug .
> 
> I think the VM option '-XX:+UnlockDiagnosticVMOptions' is unnecessary after add `@requires vm.debug == true`. Thanks for the suggest.

Yes  in the debug-build binaries  '-XX:+PrintInlining' works without '-XX:+UnlockDiagnosticVMOptions' .  If you plan to make the test work in opt too you need this additional flag. If you stay debug-only it works without it .

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

PR Comment: https://git.openjdk.org/jdk/pull/23364#issuecomment-2624727256