RFR: 8267532: Try/catch block not optimized as expected [v3]

Jorn Vernee jvernee at openjdk.org
Fri Nov 3 01:08:11 UTC 2023


On Fri, 3 Nov 2023 00:33:10 GMT, Dean Long <dlong at openjdk.org> wrote:

> We should be able to do better on the `has_monitors` issue. Loom uses this flag to short-circuit extra work when there are no ScopeDesc objects that contain monitors, so why not give Loom exactly what it wants and track this in `DebugInformationRecorder`?

I discussed the issue I ran into with Ron as well. The `has_monitors` flag is _only_ used by that one assertion (see `ContinuationHelper::CompiledFrame::is_owning_locks`). I'm not sure there is too much value in speeding up an assertion?

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

PR Comment: https://git.openjdk.org/jdk/pull/16416#issuecomment-1791763564


More information about the hotspot-dev mailing list