RFR: 8349717: [leyden] compiler/inlining/LateInlinePrinting.java is failing

duke duke at openjdk.org
Mon Feb 10 15:40:26 UTC 2025


On Mon, 10 Feb 2025 12:26:49 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> There are lots of mainline reports about this test, see links. We see a failure in current GHA for Leyden. For Leyden, there are additional specifics: we print advanced compile task information in compile log, so we get indenting/format mismatches.
> 
> Leyden dumps this:
> 
> 
>      57 W0.0 Q0.0 C0.7 3 b compiler.inlining.LateInlinePrinting$TestLateInlining::test2 (7 bytes)
>                             @ 0 compiler.inlining.LateInlinePrinting$TestLateInlining::test3 (1 bytes) inline (hot) late inline succeeded
>                             @ 3 compiler.inlining.LateInlinePrinting$TestLateInlining::testFailInline (1 bytes) failed to inline: disallowed by CompileCommand
>                             @ 6 compiler.inlining.LateInlinePrinting$TestLateInlining::testFailInline (1 bytes) failed to inline: disallowed by CompileCommand
>                             @ 9 compiler.inlining.LateInlinePrinting$TestLateInlining::test2 (7 bytes) inline (hot) late inline succeeded
>                               @ 0 compiler.inlining.LateInlinePrinting$TestLateInlining::inlined1 (1 bytes) inline (hot) late inline succeeded
>                               @ 3 compiler.inlining.LateInlinePrinting$TestLateInlining::inlined2 (1 bytes) inline (hot) late inline succeeded
>      58 W0.0 Q0.0 C1.1 4 b compiler.inlining.LateInlinePrinting$TestLateInlining::test1 (13 bytes)
> 
> 
> So, this bug fixes the test for Leyden specifically, while mainline is figuring out the rest.

@shipilev 
Your change (at version 82628e965f4cad1e8be2ac32af8b1fc057905234) is now ready to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/leyden/pull/35#issuecomment-2648428757


More information about the leyden-dev mailing list