[lworld] RFR: 8361082: [lworld] RewriteBytecodesInlineTest fails with SIGSEGV [v8]

[Coleen Phillimore]

On Wed, 9 Jul 2025 18:34:03 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> `RewriteBytecodesInlineTest` fails after # due to a new log message printing the pending exception name even though there may not be a pending exception:
>> `PENDING_EXCEPTION->klass()->name()->as_C_string()`
>> 
>> This patch refactors the loadable descriptor handling used in `SystemDictionary::load_shared_class` to better illustrate how the loadable descriptors property is handled and it corrects the log messages to be consistent with the messages used in the class file parser. Verified with tier 1-5 tests.
>
> Matias Saavedra Silva has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
> 
>  - Remove test from problemlist
>  - Merge branch 'lworld' into loadable_descriptors_crash_8361082
>  - Coleen comments
>  - Fred comments
>  - Used new method in classfileparser
>  - Coleen comments
>  - Cleanup
>  - Multilined another log message
>  - Changed long log messages to multi-line strings
>  - Added comments
>  - ... and 2 more: https://git.openjdk.org/valhalla/compare/64ba13bc...bdb81d0f

Looks good!

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

Marked as reviewed by coleenp (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/1498#pullrequestreview-3002692531